Skip to content

Added Highs::getFixedLp(HighsLp& lp)#2559

Merged
jajhall merged 12 commits into
latestfrom
fix-2557
Oct 19, 2025
Merged

Added Highs::getFixedLp(HighsLp& lp)#2559
jajhall merged 12 commits into
latestfrom
fix-2557

Conversation

@jajhall
Copy link
Copy Markdown
Member

@jajhall jajhall commented Oct 4, 2025

As with Gurobi, only generates the fixed LP. Users will have to pass the LP to HiGHS (losing the corresponding MIP) and then solve it.

@jajhall jajhall requested a review from galabovaa October 4, 2025 17:52
@jajhall jajhall marked this pull request as draft October 4, 2025 17:54
@jajhall
Copy link
Copy Markdown
Member Author

jajhall commented Oct 4, 2025

Now to add C API

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 4, 2025

Codecov Report

❌ Patch coverage is 99.59514% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.77%. Comparing base (dc852ec) to head (fd4b6f5).
⚠️ Report is 19 commits behind head on latest.

Files with missing lines Patch % Lines
check/TestMipSolver.cpp 99.04% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           latest    #2559      +/-   ##
==========================================
+ Coverage   79.71%   79.77%   +0.06%     
==========================================
  Files         346      346              
  Lines       86427    86667     +240     
==========================================
+ Hits        68891    69136     +245     
+ Misses      17536    17531       -5     

☔ 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.

@jajhall jajhall marked this pull request as ready for review October 5, 2025 21:13
@jajhall
Copy link
Copy Markdown
Member Author

jajhall commented Oct 7, 2025

This is now ready for review @galabovaa

@jajhall
Copy link
Copy Markdown
Member Author

jajhall commented Oct 7, 2025

Closes #2557

@galabovaa
Copy link
Copy Markdown
Contributor

Looks good to me! Just one conflict but it seems that we should just accept both tests

@jajhall jajhall merged commit 080579b into latest Oct 19, 2025
303 of 304 checks passed
@jajhall jajhall deleted the fix-2557 branch October 19, 2025 22:26
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