Skip to content

No cuda graph in batched pdlp - #225

Merged
rapids-bot[bot] merged 3 commits into
branch-25.08from
no_cuda_graph_batch_pdlp
Jul 23, 2025
Merged

No cuda graph in batched pdlp#225
rapids-bot[bot] merged 3 commits into
branch-25.08from
no_cuda_graph_batch_pdlp

Conversation

@Kh4ster

@Kh4ster Kh4ster commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

Temporarly disable cuda graph in batched PDLP until we find the root cause.

This allows use to re-enable the tests on the Python side

Closes #207

@Kh4ster
Kh4ster requested a review from a team as a code owner July 23, 2025 10:12
@Kh4ster
Kh4ster requested review from akifcorduk, hlinsen and kaatish and removed request for hlinsen and kaatish July 23, 2025 10:12

@akifcorduk akifcorduk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Nicolas! Could you enable the python tests again? (and cpp if there are any)

@Kh4ster Kh4ster added bug Something isn't working non-breaking Introduces a non-breaking change pdlp labels Jul 23, 2025
@Kh4ster
Kh4ster requested a review from a team as a code owner July 23, 2025 12:19
@Kh4ster
Kh4ster requested a review from tmckayus July 23, 2025 12:19
@Kh4ster
Kh4ster removed the request for review from tmckayus July 23, 2025 13:52
@Kh4ster Kh4ster self-assigned this Jul 23, 2025
@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

Temporarly disable cuda graph in batched PDLP until we find the root cause.

This allows use to re-enable the tests on the Python side

@Kh4ster PLease create an issue or tag the issue for the follow-up.

@Kh4ster

Kh4ster commented Jul 23, 2025

Copy link
Copy Markdown
Contributor Author

@rgsl888prabhu done!

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

Temporarly disable cuda graph in batched PDLP until we find the root cause.

@Kh4ster I meant a follow-up issue to enable cuda graph back with the root cause fix.

@Kh4ster

Kh4ster commented Jul 23, 2025

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit ea05ba1 into branch-25.08 Jul 23, 2025
@Kh4ster
Kh4ster deleted the no_cuda_graph_batch_pdlp branch July 23, 2025 15:16
aliceb-nv pushed a commit that referenced this pull request Sep 22, 2025
Temporarly disable cuda graph in batched PDLP until we find the root cause.

This allows use to re-enable the tests on the Python side

Closes #207

Authors:
  - Nicolas Blin (https://github.com/Kh4ster)

Approvers:
  - Akif ÇÖRDÜK (https://github.com/akifcorduk)
  - Trevor McKay (https://github.com/tmckayus)

URL: #225
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
Temporarly disable cuda graph in batched PDLP until we find the root cause.

This allows use to re-enable the tests on the Python side

Closes NVIDIA#207

Authors:
  - Nicolas Blin (https://github.com/Kh4ster)

Approvers:
  - Akif ÇÖRDÜK (https://github.com/akifcorduk)
  - Trevor McKay (https://github.com/tmckayus)

URL: NVIDIA#225
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 pdlp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Rare batched PDLP CI bug

4 participants