Skip to content

Conversation

rgsl888prabhu
Copy link
Collaborator

@rgsl888prabhu rgsl888prabhu commented Oct 13, 2025

For #476

akifcorduk and others added 28 commits October 2, 2025 08:48
Remove  unused dependency raft-dask

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

Approvers:
  - Ishika Roy (https://github.com/Iroy30)
  - Trevor McKay (https://github.com/tmckayus)

URL: NVIDIA#475
…blem (NVIDIA#477)

Fix an issue where the free variable info from the original problem was being used on the folded problem. This was leading to undefined memory access. 

Thanks to Burcin for reporting the bug and Hugo for determining the issue.

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

Approvers:
  - Hugo Linsenmaier (https://github.com/hlinsen)

URL: NVIDIA#477
This PR is a follow-up for PR NVIDIA#272 to add documentation and additional tests

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)
  - Hugo Linsenmaier (https://github.com/hlinsen)
  - Chris Maes (https://github.com/chris-maes)

Approvers:
  - Ishika Roy (https://github.com/Iroy30)
  - Trevor McKay (https://github.com/tmckayus)
  - Chris Maes (https://github.com/chris-maes)

URL: NVIDIA#449
Minor tweaks to the barrier log.

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

Approvers:
  - Hugo Linsenmaier (https://github.com/hlinsen)

URL: NVIDIA#478
This PR also fixes the starting bounds for the diving subtrees. As these nodes are detached from the main B&B tree, the bounds from the parent were lost. Now, the bounds are also kept when inserting the nodes in the diving queue.

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

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

URL: NVIDIA#474
CUDSS does not throw on OOM and even though we check for the return status. There was a sticky cuda error on any successive CUDA API call. A workaround is to throw if `cudaMallocAsync` fails in the device mem handler and catch that error in the solve method.

Authors:
  - Hugo Linsenmaier (https://github.com/hlinsen)

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

URL: NVIDIA#483
…VIDIA#484)

@tmckayus reported that in concurrent mode, he was seeing lots of barrier numerical status.

This was due to incorrectly setting the status to NUMERIC, when barrier was stopped by PDLP or concurrent.

This PR fixes the issue, by checking for concurrent limit first.  

Logs are also changed for consistency.

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

Approvers:
  - Hugo Linsenmaier (https://github.com/hlinsen)

URL: NVIDIA#484
@rgsl888prabhu rgsl888prabhu requested review from a team as code owners October 13, 2025 22:06
@rgsl888prabhu rgsl888prabhu self-assigned this Oct 13, 2025
@rgsl888prabhu rgsl888prabhu added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Oct 13, 2025
@rgsl888prabhu rgsl888prabhu changed the base branch from branch-25.10 to branch-25.12 October 14, 2025 02:38
@rgsl888prabhu rgsl888prabhu requested a review from a team as a code owner October 14, 2025 02:38
@rgsl888prabhu rgsl888prabhu requested review from Kh4ster and hlinsen and removed request for a team October 14, 2025 02:38
Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving rapids-logger update.

@rgsl888prabhu rgsl888prabhu merged commit 5636fe6 into NVIDIA:branch-25.12 Oct 15, 2025
87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants