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

Allow CuPy 11 #4837

Merged
merged 1 commit into from
Jul 28, 2022
Merged

Allow CuPy 11 #4837

merged 1 commit into from
Jul 28, 2022

Conversation

jakirkham
Copy link
Member

Allows cuML to be installed with CuPy 11.

xref: rapidsai/integration#508

@jakirkham jakirkham requested a review from a team as a code owner July 28, 2022 18:02
@github-actions github-actions bot added the conda conda issue label Jul 28, 2022
@jakirkham
Copy link
Member Author

Could someone please help add the appropriate labels here?

@dantegd dantegd added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 28, 2022
@@ -51,7 +51,7 @@ requirements:
- libcuml={{ version }}
- libcumlprims {{ minor_version }}
- pyraft {{ minor_version }}
- cupy>=7.8.0,<11.0.0a0
- cupy>=7.8.0,<12.0.0a0
Copy link
Member

Choose a reason for hiding this comment

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

Small question, should we do the same for the dev environments?

Copy link
Member Author

Choose a reason for hiding this comment

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

We can do that. Looked at the environment files here, but didn't see cupy listed. Do we want to add it as part of this PR?

Copy link
Member

Choose a reason for hiding this comment

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

I think currently it is tied to the meta packages or brought by cudf, so we should be fine with the PR as is

Copy link
Member Author

Choose a reason for hiding this comment

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

Sounds good. Happy to follow up if needed :)

@dantegd
Copy link
Member

dantegd commented Jul 28, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit dc77d6b into rapidsai:branch-22.08 Jul 28, 2022
@jakirkham jakirkham deleted the allow_cupy_11 branch July 28, 2022 23:35
@jakirkham
Copy link
Member Author

Thanks all! 🙏

galipremsagar added a commit to galipremsagar/cuml that referenced this pull request Aug 1, 2022
rapids-bot bot pushed a commit that referenced this pull request Aug 1, 2022
Reverts #4837

Authors:
  - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
  - Dante Gama Dessavre (https://github.com/dantegd)
  - Ray Douglass (https://github.com/raydouglass)

URL: #4847
jakirkham added a commit to jakirkham/cuml that referenced this pull request Feb 27, 2023
Allows cuML to be installed with CuPy 11.

xref: rapidsai/integration#508

Authors:
  - https://github.com/jakirkham

Approvers:
  - Sevag H (https://github.com/sevagh)
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: rapidsai#4837
jakirkham pushed a commit to jakirkham/cuml that referenced this pull request Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conda conda issue 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.

3 participants