Skip to content

Fix inversion of constraint bounds in conditional bounds presolve#82

Merged
rgsl888prabhu merged 5 commits intoNVIDIA:branch-25.05from
kaatish:fix_constraint_bound_inversion
Jun 9, 2025
Merged

Fix inversion of constraint bounds in conditional bounds presolve#82
rgsl888prabhu merged 5 commits intoNVIDIA:branch-25.05from
kaatish:fix_constraint_bound_inversion

Conversation

@kaatish
Copy link
Copy Markdown
Contributor

@kaatish kaatish commented Jun 8, 2025

Updating the constraint bounds based on minimal and maximal activity can sometimes cause the update constraint lower bounds to be greater than constraint upper bounds even if the constraint is feasible within tolerance.

This PR fixes the constraint bounds to the same value if this happens.

@kaatish kaatish added this to the 25.05 milestone Jun 8, 2025
@kaatish kaatish self-assigned this Jun 8, 2025
@kaatish kaatish requested a review from a team as a code owner June 8, 2025 20:54
@kaatish kaatish added bug Something isn't working non-breaking Introduces a non-breaking change labels Jun 8, 2025
@kaatish kaatish requested review from Kh4ster and rg20 June 8, 2025 20:54
@rgsl888prabhu rgsl888prabhu merged commit 5abbde3 into NVIDIA:branch-25.05 Jun 9, 2025
57 checks passed
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
…IDIA#82)

Updating the constraint bounds based on minimal and maximal activity can
sometimes cause the update constraint lower bounds to be greater than
constraint upper bounds even if the constraint is feasible within
tolerance.

This PR fixes the constraint bounds to the same value if this happens.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants