-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
It eliminated the incorrect variable, didn't it? #338
Comments
Let me add some more information. This is the variable order: julia> all_variables(m)
4-element Vector{VariableRef}:
d[1]
d[2]
g[1]
g[2] The result Polyhedron CDDLib.Polyhedron{Rational{BigInt}}:
4-element iterator of HalfSpace{Rational{BigInt}, Vector{Rational{BigInt}}}:
HalfSpace(Rational{BigInt}[-1, 0], 0//1)
HalfSpace(Rational{BigInt}[0, -1], 0//1)
HalfSpace(Rational{BigInt}[1, 0], 1//1)
HalfSpace(Rational{BigInt}[0, 1], 1//1) which is And for completeness, here is the other projection: julia> P = eliminate(p, [1:2;])
Polyhedron CDDLib.Polyhedron{Rational{BigInt}}:
4-element iterator of HalfSpace{Rational{BigInt}, Vector{Rational{BigInt}}}:
HalfSpace(Rational{BigInt}[-1, 0], -1//1)
HalfSpace(Rational{BigInt}[0, -1], -2//1)
HalfSpace(Rational{BigInt}[1, 0], 101//1)
HalfSpace(Rational{BigInt}[0, 1], 102//1) which is |
Yes, thanks. But it behaves oppositely to the example here, doesn't it? |
Pardon me, is there any change? I retested the above example, why didn't I see any changes? |
@WuSiren there is not yet a new release of this package. For now you can use the development version via the pkg> dev Polyhedra |
Oh, I see. Many thanks! 🤝 |
The result is
(false, true)
, which indicates the projection polyhedron is with respect to the eliminated variableg[1:2]
but not the expectedd[1:2]
.Is this a bug, or I didn't use it correctly?
Looking forward to response. Thanks!
The text was updated successfully, but these errors were encountered: