-
Notifications
You must be signed in to change notification settings - Fork 915
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
Use PyNVML 12 #17627
Use PyNVML 12 #17627
Conversation
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
/ok to test |
Will need to update to PyNVML 12 in Dask-CUDA before this will work. CI failures underline this point xref: rapidsai/dask-cuda#1419 |
Marking as ready for review. Will rerun CI after the Dask-CUDA PR lands |
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.
Makes sense to me based on the other related changes, but I'd like to see CI results before approving.
All good. Wanted to see a rerun as well Mainly stating that for context In any event Dask-CUDA has rebuilt with those changes So have pulled in the latest cuDF changes to refresh the PR and rerun CI. Let's see how it goes |
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.
LGTM, thanks @jakirkham . Let's get this merged ASAP, as this is blocking cuDF's nightly build-docs.
/merge |
f7f7e33
into
rapidsai:branch-25.02
Description
Bump
pynvml
from11
to12
. This version ofpynvml
also now depends onnvidia-ml-py
for core functionality.Depends on PR: rapidsai/dask-cuda#1419
Checklist