-
Notifications
You must be signed in to change notification settings - Fork 27
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
Breaking changes for MOI 0.10 #219
Conversation
Codecov Report
@@ Coverage Diff @@
## master #219 +/- ##
==========================================
- Coverage 87.32% 85.27% -2.05%
==========================================
Files 8 7 -1
Lines 418 292 -126
==========================================
- Hits 365 249 -116
+ Misses 53 43 -10
Continue to review full report at Codecov.
|
Okay, after this experience, I think I'm now on board with:
All-in-all, this rewrite of SCS.jl was not a fun experience, but I think we're in a much better place than we were. |
@blegat did you actually try jump-dev/MathOptInterface.jl#1592 on a solver? It isn't working. |
julia> model = MOI.Bridges.full_bridge_optimizer(
MOI.Utilities.CachingOptimizer(
MOI.Utilities.UniversalFallback(MOI.Utilities.Model{Float64}()),
SCS.Optimizer(),
),
Float64,
);
julia> MOI.set(model, MOI.Silent(), true)
julia> x, ci = MOI.add_constrained_variables(model, MOI.Nonnegatives(2));
julia> MOI.optimize!(model)
julia> MOI.empty!(model)
julia> MOI.get(model, MOI.TerminationStatus())
OPTIMAL::TerminationStatusCode = 1 |
More test failures than we should be happy with. I'll comment in-line.