Skip to content

fix: use HiGHS to parse CBC sol correctly#442

Merged
FabianHofmann merged 3 commits intomasterfrom
cbc-parse-sol
Apr 15, 2025
Merged

fix: use HiGHS to parse CBC sol correctly#442
FabianHofmann merged 3 commits intomasterfrom
cbc-parse-sol

Conversation

@siddharth-krishna
Copy link
Copy Markdown
Collaborator

Closes # (if applicable).

Changes proposed in this Pull Request

CBC's solve_problem_from_file method was assuming that all variables start with x which is not true when calling the solver on a pre-existing MPS file. This PR uses highspy to parse the input file and get the set of variable names, which can then be used as a more accurate filter when parsing CBC's solution file.

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in doc.
  • Unit tests for new features were added (if applicable).
  • A note for the release notes doc/release_notes.rst of the upcoming release is included.
  • I consent to the release of this PR's code under the MIT license.

@FabianHofmann FabianHofmann merged commit bbca627 into master Apr 15, 2025
2 of 4 checks passed
@FabianHofmann FabianHofmann deleted the cbc-parse-sol branch April 15, 2025 20:04
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.

2 participants