Skip to content

Add support for starting values#208

Draft
blegat wants to merge 8 commits intomasterfrom
bl/start_again
Draft

Add support for starting values#208
blegat wants to merge 8 commits intomasterfrom
bl/start_again

Conversation

@blegat
Copy link
Member

@blegat blegat commented Mar 11, 2026

Continuation for #166

This also fixes a bug: the result_index were ignored and we would invariably query the first solution.

  • Test for starting values
  • Test for the result_index bug
  • Constrained variables

Closes #165

We were discussing this with @SophieL1. We have a use case (repo not public yet) where we need to set and get constraint attributes of a solver that is behind a dualization layer. It would be nice to have clearly defined dual_attribute, ... that allows to define how the attributes should be transformed through a dualization layer.

@blegat blegat mentioned this pull request Mar 11, 2026
3 tasks
@blegat blegat marked this pull request as draft March 11, 2026 14:28
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 48.68421% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.90%. Comparing base (c347f54) to head (cce23b4).

Files with missing lines Patch % Lines
src/MOI_wrapper.jl 48.68% 39 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #208      +/-   ##
==========================================
- Coverage   94.20%   89.90%   -4.31%     
==========================================
  Files          14       14              
  Lines         742      812      +70     
==========================================
+ Hits          699      730      +31     
- Misses         43       82      +39     

☔ View full report in Codecov by Sentry.
📢 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.

Comment on lines +16 to +18

[sources]
Dualization = {path = ".."}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needed?

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Support starting values

2 participants