-
Notifications
You must be signed in to change notification settings - Fork 15
Work with the changes in the output format of MibS and add test examples for MibS solver #101
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
base: master
Are you sure you want to change the base?
Conversation
Hey, is there any progress here to merge the PR? I would like to utilise the mibs functionality |
@whart222 This looks pretty good, what do you think? |
@tkralphs I think I will pull this branch and continue to work on my machine. One problem related question; I want to use MiBS to solve a bi-level problem with lower-level and upper-level MILP (non-separable variables), does that work ? |
@VioletIsSoCute So to clarify, this PR is not working for you? This is supposed to fix #98. I have not personally tried using it yet, but it looks correct and seems to be working for @Paulnkk. I can try to look into it if it seems to be broken. I am hoping that @whart222 can take a look at and merge this PR at some point. |
@tkralphs Yeah, this thread highlighted that I had dropped the ball on doing this merge. I'll try to get this done in the next few weeks. |
@VioletIsSoCute I guess my question was whether this PR is not working for you. The issue was reported in #98, but this PR was supposed to have fixed it and you seem to be saying that it didn't fix it for you. Is that correct? If so, it might help to say more about what went wrong. If you haven't tried to use this PR, then that might be all you need. |
@tkralphs After posting the question, I tried to reinstall the MibS solver using coinbrew. However, there is some trouble for running Clp unit test, with error message as bellow:################################################## Configuring Clp master################################################## ################################################## Building Clp master################################################## ################################################## Running Clp unit test################################################## Testing ClpSimplex Error: Build failed, see error output aboveI'm not sure what is happening and could not find a fix for it. Not even sure about the first downloaded version is working or not too (and I deleted it), because I probably ignore the error message. |
Hmm, that is odd, but you can safely skip the test with the argument |
This PR tries to make PAO work with MibS as pointed out in #98.
And raise the error if the linking variables are not integer: "ERROR:All linking variables should be integer"
@mkoeppe