Skip to content

Conversation

@rapids-bot
Copy link
Contributor

@rapids-bot rapids-bot bot commented Nov 21, 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.

cuvs was required to create test dataset and it was used by cupy internally, but since we are doing it on cpu now, removing this dependency.

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Jake Awe (https://github.com/AyodeAwe)
  - Trevor McKay (https://github.com/tmckayus)

URL: #615
@rapids-bot rapids-bot bot requested review from a team as code owners November 21, 2025 15:57
@rapids-bot
Copy link
Contributor Author

rapids-bot bot commented Nov 21, 2025

FAILURE - Unable to forward-merge due to an error, manual merge is necessary. Do not use the Resolve conflicts option in this PR, follow these instructions https://docs.rapids.ai/maintainers/forward-merger/

IMPORTANT: When merging this PR, do not use the auto-merger (i.e. the /merge comment). Instead, an admin must manually merge by changing the merging strategy to Create a Merge Commit. Otherwise, history will be lost and the branches become incompatible.

rgsl888prabhu and others added 2 commits November 21, 2025 17:42
Adds missing LP and MILP settings

## Issue

closes #595

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Rajesh Gandham (https://github.com/rg20)
  - Trevor McKay (https://github.com/tmckayus)

URL: #613
With this PR, the solver propagates the bounds from the parent to the child nodes. If the solver reaches the end of a branch (i.e., when the node becomes infeasible or an integer feasible solution is found), the bounds are recomputed from scratch. This allows the solver to reuse the tightened bounds from the parent and re-apply the bounds strengthening on the branched variable.


Average (Primal) Gap over the MIPLIB2017 dataset: 
`main` (10f116b): `14.472358` with `224` feasible solutions
This PR: `14.058718` with `226` feasible solutions

On average, the `main` branch explored `1012412` nodes. This PR increases to `1607821`  nodes (i.e., `58%` more nodes) for the same time frame.

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

Approvers:
  - Rajesh Gandham (https://github.com/rg20)
  - Chris Maes (https://github.com/chris-maes)
  - Akif ÇÖRDÜK (https://github.com/akifcorduk)
  - Ramakrishnap (https://github.com/rgsl888prabhu)

URL: #473
@rapids-bot rapids-bot bot requested a review from a team as a code owner November 24, 2025 08:40
@rapids-bot rapids-bot bot requested review from hlinsen and kaatish November 24, 2025 08:40
@rgsl888prabhu rgsl888prabhu mentioned this pull request Nov 24, 2025
8 tasks
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