Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Condition cudatoolkit in test/requires on CUDA 11 #13613

Closed
jakirkham opened this issue Jun 23, 2023 · 1 comment · Fixed by #13615
Closed

Condition cudatoolkit in test/requires on CUDA 11 #13613

jakirkham opened this issue Jun 23, 2023 · 1 comment · Fixed by #13615

Comments

@jakirkham
Copy link
Member

Currently cudatoolkit is added to test/requires independent of CUDA version. Think we may want to make this conditional / limited to CUDA 11. Alternatively this could use cuda-version

cudf/conda/recipes/cudf/meta.yaml

Lines 103 to 105 in 0b4e354

test:
requires:
- cudatoolkit ={{ cuda_version }}

@jakirkham jakirkham mentioned this issue Jun 23, 2023
Merged
@jakirkham
Copy link
Member Author

cc @bdice

@bdice bdice mentioned this issue Jun 26, 2023
Merged
3 tasks
rapids-bot bot pushed a commit that referenced this issue Jun 26, 2023
@bdice
Use cuda-version to constrain cudatoolkit. (#13615)
cdb08fc 
This PR changes CUDA 11 packaging to rely on `cuda-version` pinnings to constrain the installed version of `cudatoolkit`.

Resolves #13613. Resolves #13607.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Ray Douglass (https://github.com/raydouglass)
  - https://github.com/jakirkham

URL: #13615
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
cuDF/Dask/Numba/UCX
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use cuda-version to constrain cudatoolkit. bdice/cudf
1 participant
@jakirkham