Conversation
|
The Juniper and Pavito failures are super interesting and they are caused by #2857
Maybe we should revert #2857 and add a check at the JuMP level? Or maybe the bridge needs to say: I added a Or maybe we need to convert the |
|
|
Hearing no objections? |
Pavito and Juniper shouldn't support Indicator constraints so that it's bridged before they pass it down. It doesn't make sense for them to pass down an indicator constraints to a non-MIP solver. It's not like there is a chance for these non-MIP solvers to support indicator constraints. |
|
Maybe this should be fixed before we release ? |
|
Already done in #2868 |
|
Latest failures in Alpine and PowerModels are because of jump-dev/HiGHS.jl#302 |
|
@blegat can you confirm you're okay with my changes with the ToMILPBridge? |
Ok so the issue was that Juniper said that it supported indicator because it was redirecting to the inner solver which said it was supporting it. Now with https://github.com/jump-dev/MathOptInterface.jl/pull/2868/files, the inner solver says it does not support indicator so Juniper says the same so it is bridged in the bridge outer of Juniper, that's an elegant fix. |
Yip, although it isn't a perfect fix. On the outside, Juniper fixes |
Basic
versionfield ofProject.tomlhas been updated- If a breaking change, increment the MAJOR field and reset others to 0
- If adding new features, increment the MINOR field and reset PATCH to 0
- If adding bug fixes or documentation changes, increment the PATCH field
Documentation
docs/src/changelog.md, following existing styleTests
https://github.com/jump-dev/MathOptInterface.jl/actions/runs/18326452732https://github.com/jump-dev/MathOptInterface.jl/actions/runs/18362251645MOI.Test.version_addedis implemented.