Skip to content

Build and test with CUDA 13.0.0#466

Merged
rapids-bot[bot] merged 8 commits intorapidsai:branch-25.10from
jameslamb:cuda-13.0.0
Sep 2, 2025
Merged

Build and test with CUDA 13.0.0#466
rapids-bot[bot] merged 8 commits intorapidsai:branch-25.10from
jameslamb:cuda-13.0.0

Conversation

@jameslamb
Copy link
Member

Contributes to rapidsai/build-planning#208

  • uses CUDA 13.0.0 to build and test
  • adds CUDA 13 devcontainers
  • moves some dependency pins:
    • cuda-python: >=12.9.2 (CUDA 12), >=13.0.1 (CUDA 13)
    • cupy: >=13.6.0

Contributes to rapidsai/build-planning#68

  • updates to CUDA 13 dependencies in fallback entries in dependencies.yaml matrices (i.e., the ones that get written to pyproject.toml in source control)

Notes for Reviewers

This switches GitHub Actions workflows to the cuda13.0 branch from here: rapidsai/shared-workflows#413

A future round of PRs will revert that back to branch-25.10, once all of RAPIDS supports CUDA 13.

@jameslamb jameslamb added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Aug 26, 2025
@copy-pr-bot
Copy link

copy-pr-bot bot commented Aug 26, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@jameslamb
Copy link
Member Author

/ok to test

PYTHON_WHEELHOUSE=$(RAPIDS_PY_WHEEL_NAME="rapidsmpf_${RAPIDS_PY_CUDA_SUFFIX}" rapids-download-wheels-from-github python)

# generate test dependencies
rapids-dependency-file-generator --output requirements --file-key "test_python" --matrix "cuda=12.*;arch=x86_64;cuda_suffixed=true;py=313" | tee -a "$PIP_CONSTRAINT"
Copy link
Member Author

@jameslamb jameslamb Aug 28, 2025

Choose a reason for hiding this comment

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

I first looked at this line because the addition of dask-cuda[cu12] caused this error from pip install:

ERROR: Constraints cannot have extras

(build link)

On closer inspection, I think this line is broken and also unnecessary:

  1. nothing in dependencies.yaml would match py=313 (the py: entries are spelled with dots like py: "3.13")
  2. everything in test_python already ends up in the [test] extra of the wheels

Just removing it simplifies this script and avoids the need to update those hard-coded CUDA version, CPU architecture, and Python version as the RAPIDS support matrix changes.

@jameslamb
Copy link
Member Author

Wheels are looking good! But looks like we'll need ucc packages on conda-forge with CUDA 13 support:

│  ├─ openmpi 5.0.8 would require
│  │  └─ ucc >=1.4.4,<2.0a0 * but there are no viable options
│  │     ├─ ucc 1.4.4 would require
│  │     │  └─ cuda-version >=11,<12.0a0 *, which conflicts with any installable versions previously reported;
│  │     └─ ucc [1.4.4|1.5.0] would require
│  │        └─ cuda-version >=12,<13.0a0 *, which conflicts with any installable versions previously reported;

(conda-cpp-build link)

Thankfully @marcinz is working on that here: conda-forge/ucc-feedstock#31

@jameslamb
Copy link
Member Author

/ok to test

@jameslamb jameslamb changed the title WIP: Build and test with CUDA 13.0.0 Build and test with CUDA 13.0.0 Sep 2, 2025
@jameslamb jameslamb marked this pull request as ready for review September 2, 2025 04:18
@jameslamb jameslamb requested review from a team as code owners September 2, 2025 04:18
@jameslamb jameslamb requested review from AyodeAwe and bdice and removed request for bdice September 2, 2025 04:18
@jameslamb
Copy link
Member Author

Thanks for the review @AyodeAwe ! Before we merge this, I'd like one of the library developers working on rapidsmpf to also review this set of changes. I'll go ask for that.

Copy link
Member

@pentschev pentschev left a comment

Choose a reason for hiding this comment

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

LGTM, thanks James!

@jameslamb
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit 012d1da into rapidsai:branch-25.10 Sep 2, 2025
176 of 178 checks passed
@jameslamb jameslamb deleted the cuda-13.0.0 branch September 2, 2025 16:27
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.

3 participants