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

Update DEFAULT_CUDA_VER in ci/cpu/prebuild.sh #9749

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

ajschmidt8
Copy link
Member

Now that we only do 11.5 builds for RAPIDS, the DEFAULT_CUDA_VER variable in ci/cpu/prebuild.sh should be set to 11.5 so that the rest of the logic in the file works correctly.

Now that we only do `11.5` builds for RAPIDS, the `DEFAULT_CUDA_VER` variable in `ci/cpu/prebuild.sh` should be set to `11.5` so that the rest of the logic in the file works correctly.
@ajschmidt8 ajschmidt8 added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 22, 2021
@ajschmidt8 ajschmidt8 requested a review from a team as a code owner November 22, 2021 14:47
@github-actions github-actions bot added the gpuCI label Nov 22, 2021
@bdice
Copy link
Contributor

bdice commented Nov 22, 2021

@ajschmidt8 This changeset looks fine but does it need to target 21.12? cc: @beckernick

@ajschmidt8 ajschmidt8 changed the base branch from branch-22.02 to branch-21.12 November 22, 2021 15:18
@beckernick
Copy link
Member

Yep, this should target 21.12

@raydouglass
Copy link
Member

rerun tests

1 similar comment
@raydouglass
Copy link
Member

rerun tests

@codecov
Copy link

codecov bot commented Nov 22, 2021

Codecov Report

Merging #9749 (7d7053c) into branch-21.12 (ab4bfaa) will decrease coverage by 0.17%.
The diff coverage is n/a.

❗ Current head 7d7053c differs from pull request most recent head f24c7be. Consider uploading reports for the commit f24c7be to get more accurate results
Impacted file tree graph

@@               Coverage Diff                @@
##           branch-21.12    #9749      +/-   ##
================================================
- Coverage         10.79%   10.61%   -0.18%     
================================================
  Files               116      118       +2     
  Lines             18869    20072    +1203     
================================================
+ Hits               2036     2130      +94     
- Misses            16833    17942    +1109     
Impacted Files Coverage Δ
python/dask_cudf/dask_cudf/sorting.py 92.90% <0.00%> (-1.21%) ⬇️
python/cudf/cudf/io/csv.py 0.00% <0.00%> (ø)
python/cudf/cudf/io/hdf.py 0.00% <0.00%> (ø)
python/cudf/cudf/io/orc.py 0.00% <0.00%> (ø)
python/cudf/cudf/__init__.py 0.00% <0.00%> (ø)
python/cudf/cudf/_version.py 0.00% <0.00%> (ø)
python/cudf/cudf/core/abc.py 0.00% <0.00%> (ø)
python/cudf/cudf/core/cut.py 0.00% <0.00%> (ø)
python/cudf/cudf/api/types.py 0.00% <0.00%> (ø)
python/cudf/cudf/io/dlpack.py 0.00% <0.00%> (ø)
... and 71 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ff5d987...f24c7be. Read the comment docs.

@ajschmidt8
Copy link
Member Author

rerun tests

@ajschmidt8
Copy link
Member Author

The CI failures are unrelated to this PR. admin merging to unblock some things downstream.

@ajschmidt8 ajschmidt8 merged commit 9fc35b7 into rapidsai:branch-21.12 Nov 22, 2021
@ajschmidt8 ajschmidt8 deleted the update-cuda-ver branch November 22, 2021 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants