-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor dependency-file-generator handling of cupy, default to CUDA 12 in fallback dependency lists #1059
Conversation
- cupy>=12.0.0 | ||
# NOTE: This is intentionally not broken into groups by a 'cuda_suffixed' selector like | ||
# other packages with -cu{nn}x suffixes in this file. | ||
# All RAPIDS wheel builds (including in devcontainers) expect cupy to be suffixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is exactly the language I'm proposing over in rapidsai/cugraph#4552 (comment) as well
/merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks James! 🙏
- | | ||
pip install \ | ||
-C rapidsai.matrix-entry="cuda=12.2;cuda_suffixed=true" \ | ||
-C rapidsai.matrix-entry="cuda=12.x;cuda_suffixed=true" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! 😄
Follow-up to #1057.
Implements the following suggestions based on review comments from @jakirkham and @bdice there:
cupy
out into its own list independencies.yaml
, to allow removing somecuda_suffixed: "false"
blocks that were otherwise redundant--matrix-entry 'cuda=12.x'
in docs builds, to make it clearer that those builds are just trying to matchcuda: "12.*"
and are not at all dependent on any particular minor version of CUDAAlso proposes defaulting to
cupy-cuda12x
inpyproject.toml
, as part of rapidsai/build-planning#68.