-
Notifications
You must be signed in to change notification settings - Fork 913
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
Add pynvml
as a dependency for dask-cudf
#17386
Add pynvml
as a dependency for dask-cudf
#17386
Conversation
Thanks Peter! 🙏 Do we want to drop this cudf/python/dask_cudf/dask_cudf/io/parquet.py Lines 43 to 60 in 332cc06
|
The |
Removed the |
@@ -43,6 +43,7 @@ requirements: | |||
run: | |||
- python | |||
- cudf ={{ version }} | |||
- pynvml >=11.4.1,<12.0.0a0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pushed this small change to include pynvml
in the dask-cudf
Conda package. Hope that is ok 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Peter and Rick! 🙏
/merge |
Description
#17250 started using
pynvml
but did not add the proper dependency, this change fixes the missing dependency.Checklist