Skip to content

Add support for cuda13 container and fix cuda13 lib issues in wheel - #379

Merged
rapids-bot[bot] merged 6 commits into
NVIDIA:branch-25.10from
ramakrishnap-nv:support_cuda13_container
Sep 9, 2025
Merged

Add support for cuda13 container and fix cuda13 lib issues in wheel#379
rapids-bot[bot] merged 6 commits into
NVIDIA:branch-25.10from
ramakrishnap-nv:support_cuda13_container

Conversation

@ramakrishnap-nv

@ramakrishnap-nv ramakrishnap-nv commented Sep 8, 2025

Copy link
Copy Markdown
Collaborator

Description

  • This PR adds support for cuda 13 container for nightly and release
  • Also updates python version to 3.13
  • Also cuda-toolkits have stopped using cu13 suffixes and they need to be fixed in dependencies
  • This also requires addition to RPATH since the paths are different for cu13

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue to follow-up
    • NA
  • Documentation
    • The documentation is up to date with these changes
    • Added new documentation
    • NA

@ramakrishnap-nv
ramakrishnap-nv requested review from a team as code owners September 8, 2025 17:08
@ramakrishnap-nv ramakrishnap-nv self-assigned this Sep 8, 2025
@ramakrishnap-nv ramakrishnap-nv added bug Something isn't working non-breaking Introduces a non-breaking change labels Sep 8, 2025
@ramakrishnap-nv ramakrishnap-nv added this to the 25.10 milestone Sep 8, 2025

@jameslamb jameslamb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You'll need to update some of these as well:

cuopt/dependencies.yaml

Lines 756 to 764 in 3188bda

- matrix:
cuda: "13.*"
use_cuda_wheels: "true"
packages:
- nvidia-cublas-cu13
- nvidia-curand-cu13
- nvidia-cusparse-cu13
- nvidia-cusolver-cu13
- nvidia-nvtx-cu13

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

You'll need to update some of these as well:

cuopt/dependencies.yaml

Lines 756 to 764 in 3188bda

- matrix:
cuda: "13.*"
use_cuda_wheels: "true"
packages:
- nvidia-cublas-cu13
- nvidia-curand-cu13
- nvidia-cusparse-cu13
- nvidia-cusolver-cu13
- nvidia-nvtx-cu13

Thought I had pushed, but some how missed it. Thank you for pointing it out @jameslamb .

@ramakrishnap-nv ramakrishnap-nv changed the title Add support for cuda13 and fix cuda13 lib issues in wheel Add support for cuda13 container and fix cuda13 lib issues in wheel Sep 8, 2025

@jameslamb jameslamb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for catching this today and prompting discussion that led me to learn a lot of things. I think several wheel workflows here could be simplified by switching to the cuda-toolkit metapackage.

Left some comments about that. Marking this "approve" just so you don't need to wait for another review from me to merge, but I hope you'll consider implementing those suggestions within this PR.

Comment thread dependencies.yaml Outdated
Comment thread ci/docker/Dockerfile Outdated
@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

/merge

@rapids-bot
rapids-bot Bot merged commit 22bec1f into NVIDIA:branch-25.10 Sep 9, 2025
102 checks passed
aliceb-nv pushed a commit that referenced this pull request Sep 22, 2025
…379)

- This PR adds support for cuda 13 container for nightly and release
- Also updates python version to 3.13
- Also cuda-toolkits have stopped using cu13 suffixes and they need to be fixed in dependencies
- This also requires addition to RPATH since the paths are different for cu13

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

Approvers:
  - Trevor McKay (https://github.com/tmckayus)
  - James Lamb (https://github.com/jameslamb)

URL: #379
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
…VIDIA#379)

- This PR adds support for cuda 13 container for nightly and release
- Also updates python version to 3.13
- Also cuda-toolkits have stopped using cu13 suffixes and they need to be fixed in dependencies
- This also requires addition to RPATH since the paths are different for cu13

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

Approvers:
  - Trevor McKay (https://github.com/tmckayus)
  - James Lamb (https://github.com/jameslamb)

URL: NVIDIA#379
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.

3 participants