wheels-build: build on CUDA 13.0#510
Conversation
Co-authored-by: Bradley Dice <bdice@bradleydice.com>
jameslamb
added
breaking
jameslamb
changed the title
There was a problem hiding this comment.
Should we change the latest tag of ci-wheel too?
edit: it probably doesn't matter, it's not widely used. https://github.com/search?q=org%3Arapidsai%20%2Fci-wheel.*latest%2F&type=code
|
I think it's still a good idea to update But also agree it's not on the critical path for this, so we can just put it up on the side. I'll do that. Thanks for thinking about that connection, I hadn't. |
|
/merge |
|
@bdice I put up a PR to change the I sometimes do |
|
I've deleted this branch, to be sure I caught all the references in PRs. |
Fixes #374 #373 added CUDA 13.0.2 and 12.2.2 `ci-wheel` images, for use in experimenting with building RAPIDS against older CTKs. The 13.0.2 images must be kept for now (rapidsai/build-planning#257), but the 12.2.2 images aren't needed. This: * excludes `ci-wheel` CUDA 12.2.2 images * moves `ci-wheel:{rapids-version}-latest` to CUDA 13.0.2, to align with how we're building wheels as of rapidsai/build-planning#257 - _NOTE: only really matters in CI for the next time we got to build `libucx` wheels (ref: rapidsai/shared-workflows#510 (review)), but still helpful for quick local debugging / CI reproductions_ Authors: - James Lamb (https://github.com/jameslamb) Approvers: - Bradley Dice (https://github.com/bdice) URL: #386
3 participants
Contributes to rapidsai/build-planning#257
Proposes switching CUDA 13 wheel builds to using CTK 13.0, to avoid pulling in a dependency on
nvjitlink >= 13.1. See the linked issue for details.Notes for Reviewers
How I tested this
rmm,kvikio,ucxx,raft,cuvs,cudf, andcugraphpointed at this branch and using each others' CI artifacts and testing against a range of CTK versions (wheels CI: test mix ofcuda-toolkitversion in CI build-planning#256) ... did not see any issuesWhen can we merge this?
I think it's safe to merge immediately.
The PRs in all the repos update dependency pins and add testing against different CTK versions, but PR CI should still work on
release/26.04in all of them immediately after merging this.