-
Notifications
You must be signed in to change notification settings - Fork 18
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
varEq on a BinVar is ignored. #10
Comments
Looking into this further ( |
I must say I don't really understand your issue. Further I have not used
this package for a couple of years so your guess is probably as good as
mine.
…On Wed, Aug 1, 2018 at 4:05 PM, justinpastoncooper ***@***.*** > wrote:
Looking into this further (Data.LinearProgram.GLPK.Internal), this seems
to be more of a feature of GLPK. What's your opinion on this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#10 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AABsY4DzDop1NM_TkhXUZL9Ml9ozoNmdks5uMbWzgaJpZM4Vqhz7>
.
|
Maybe I didn't explain in the best way. |
Thanks for documenting the issue. Perhaps you'll want to push this upstream. |
Sorry, I only just saw your message. Maybe it's better to retain the current behaviour so that it mirrors the glpk library? |
This can either be fixed upstream, or we can add a workaround (for example by adding a prefix to all variables). In the latter case I'll be happy to get a pull request. |
I sadly don't have access to the computer I wrote the fix on right now. I will get back to this when I do. |
Suppose we have an
LP v c
withsetVarKind x BinVar
at the top. Later on, I might like to setvarEq x 0
or even to1
, but never a non-binary value, to rulex
out or make it certain under chosen conditions. Currently thisvarEq
is ignored and the only constraint that goes through to the linear programme is that0 <= x <= 1
and thatx
is general.The text was updated successfully, but these errors were encountered: