Skip to content

Fix dual_objective_value error from MOI - #446

Merged
pulsipher merged 1 commit into
infiniteopt:masterfrom
dnguyen227:mock_dual_objective
Sep 25, 2026
Merged

pulsipher merged 1 commit into
infiniteopt:masterfrom
dnguyen227:mock_dual_objective

Conversation

@dnguyen227

@dnguyen227 dnguyen227 commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Test-only fix for the dual_objective_value test in test/results.jl, which fails with MathOptInterface v1.54.0 (released 2026-09-22):

MathOptInterface.GetAttributeNotAllowed{MathOptInterface.DualObjectiveValue}:
Cannot get the dual objective with an objective function of type `MathOptInterface.ScalarQuadraticFunction{Float64}`

Patch notes here:
https://github.com/jump-dev/MathOptInterface.jl/releases/tag/v1.54.0
PR causing error here:
jump-dev/MathOptInterface.jl#3086

@codecov

codecov Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.72%. Comparing base (467c671) to head (7ec20a1).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #446   +/-   ##
=======================================
  Coverage   99.72%   99.72%           
=======================================
  Files          39       39           
  Lines        6830     6830           
=======================================
  Hits         6811     6811           
  Misses         19       19           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dnguyen227

Copy link
Copy Markdown
Member Author

@pulsipher ready for review.

@pulsipher
pulsipher merged commit fc6256d into infiniteopt:master Sep 25, 2026
9 checks passed
@odow

odow commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Yeah sorry. This was a bug fix. Previous code incorrect

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants