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.

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

Constrain CUDA in notebook testing to prevent CUDA 12.1 usage until we have pynvjitlink #14648

Merged
merged 1 commit into from
Dec 18, 2023

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Dec 18, 2023

Description

Our notebook tests will currently fail to run because CUDA 12.1 is now available on conda-forge. We need to constrain the testing environment (we already do this for non-notebook testing environments).

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@vyasr vyasr added bug Something isn't working non-breaking Non-breaking change labels Dec 18, 2023
@vyasr vyasr self-assigned this Dec 18, 2023
@vyasr vyasr marked this pull request as ready for review December 18, 2023 20:55
@vyasr vyasr requested a review from a team as a code owner December 18, 2023 20:55
@vyasr vyasr changed the title Constraint CUDA in notebook testing to prevent CUDA 12.1 usage until we have pynvjitlink Constrain CUDA in notebook testing to prevent CUDA 12.1 usage until we have pynvjitlink Dec 18, 2023
@vyasr
Copy link
Contributor Author

vyasr commented Dec 18, 2023

/merge

@rapids-bot rapids-bot bot merged commit bb047a2 into rapidsai:branch-24.02 Dec 18, 2023
68 checks passed
@vyasr vyasr deleted the fix/constrain_cuda_version branch December 18, 2023 22:51
rapids-bot bot pushed a commit that referenced this pull request Jan 9, 2024
Follow up to #14648

Since `rapidsai/docker` tests the notebooks on both CUDA 11.8 & 12.0, we need a CUDA 11.8 version of the `test_notebooks` file key.

Without this change rdfg wlll always generate an environment with `cuda-version=12.0` even when the matrix is for CUDA 11.8 and then it will conflict with other repos (as see [here](https://github.com/rapidsai/docker/actions/runs/7451820911/job/20273765522#step:10:316)).

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

Approvers:
  - AJ Schmidt (https://github.com/ajschmidt8)
  - Bradley Dice (https://github.com/bdice)

URL: #14722
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 Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants