Skip to content

Conversation

@rapids-bot
Copy link
Contributor

@rapids-bot rapids-bot bot commented Dec 10, 2025

Forward-merge triggered by push to release/25.12 that creates a PR to keep main up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge. See forward-merger docs for more info.

…ranching (#678)

If the LP relaxation when branching on a fractional variable in the strong branching do not lead to a optimal solution and the iteration limit is not reach, then the corresponding pseudocost is set to infinite. This ignore any issue in the dual simplex and cause the pseudocost average to also be infinite.

This PR properly handle numerical issues in the dual simplex in strong branching and exclude infinite pseudocost to be included in the average.

Not critical for 25.12.

Authors:
  - Nicolas L. Guidotti (https://github.com/nguidotti)

Approvers:
  - Chris Maes (https://github.com/chris-maes)

URL: #678
@rapids-bot rapids-bot bot requested a review from a team as a code owner December 10, 2025 04:24
@rapids-bot rapids-bot bot requested a review from kaatish December 10, 2025 04:24
@GPUtester GPUtester merged commit 14ef730 into main Dec 10, 2025
@rapids-bot rapids-bot bot requested a review from hlinsen December 10, 2025 04:24
@rapids-bot
Copy link
Contributor Author

rapids-bot bot commented Dec 10, 2025

SUCCESS - forward-merge complete.

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