Skip to content

[REVIEW] Change minimum pin of cupy#9636

Merged
ajschmidt8 merged 1 commit intorapidsai:branch-21.12from
galipremsagar:cupy_min_pin
Nov 18, 2021
Merged

[REVIEW] Change minimum pin of cupy#9636
ajschmidt8 merged 1 commit intorapidsai:branch-21.12from
galipremsagar:cupy_min_pin

Conversation

@galipremsagar
Copy link
Contributor

This PR updates minimum version pinning of cupy that is compatible with CEC.

@galipremsagar galipremsagar self-assigned this Nov 9, 2021
@galipremsagar galipremsagar requested a review from a team as a code owner November 9, 2021 03:47
@github-actions github-actions bot added the conda label Nov 9, 2021
@galipremsagar galipremsagar added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change and removed conda labels Nov 9, 2021
@codecov
Copy link

codecov bot commented Nov 9, 2021

Codecov Report

Merging #9636 (7fccecb) into branch-21.12 (ab4bfaa) will decrease coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           branch-21.12    #9636      +/-   ##
================================================
- Coverage         10.79%   10.66%   -0.13%     
================================================
  Files               116      117       +1     
  Lines             18869    19832     +963     
================================================
+ Hits               2036     2115      +79     
- Misses            16833    17717     +884     
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/api/types.py 0.00% <0.00%> (ø)
python/cudf/cudf/io/dlpack.py 0.00% <0.00%> (ø)
python/cudf/cudf/core/frame.py 0.00% <0.00%> (ø)
... and 67 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 3280be2...7fccecb. Read the comment docs.

Copy link
Member

@ajschmidt8 ajschmidt8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's hold off on merging this while I debug some things from rapidsai/integration#388. Some of the failures there are expected, but some of them aren't.

@ajschmidt8 ajschmidt8 merged commit d623c93 into rapidsai:branch-21.12 Nov 18, 2021
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.

2 participants