Skip to content

Conversation

jameslamb
Copy link
Member

@jameslamb jameslamb commented Sep 2, 2025

Contributes to https://github.com/rapidsai/build-infra/issues/293

The pynvml package (https://github.com/gpuopenanalytics/pynvml) provides 2 things:

  • the pynvml Python module, transitively via a dependency on the nvidia-ml-py package
  • the pynvml_utils Python module

This project's doesn't need the pynvml_utils Python module, so this PR proposes dropping the dependency on the pynvml package in favor of nvidia-ml-py.

Notes for Reviewers

How I tested this

Inspired by rapidsai/dask-cuda#1547 (comment), checked the runtime environment for tests... saw that pynvml was not installed.

e.g. https://github.com/rapidsai/jupyterlab-nvdashboard/actions/runs/17393269109/job/49370884605?pr=242

@jameslamb jameslamb added DO NOT MERGE Hold off on merging; see PR for details non-breaking improvement and removed DO NOT MERGE Hold off on merging; see PR for details labels Sep 2, 2025
@jameslamb jameslamb changed the title WIP: use 'nvidia-ml-py' package for 'pynvml' module use 'nvidia-ml-py' package for 'pynvml' module Sep 2, 2025
@jameslamb jameslamb marked this pull request as ready for review September 2, 2025 14:44
@jameslamb jameslamb requested a review from a team as a code owner September 2, 2025 14:44
Copy link
Member

@pentschev pentschev left a comment

Choose a reason for hiding this comment

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

LGTM.

@jameslamb
Copy link
Member Author

Thank you both!

@jameslamb
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit 424cc8d into rapidsai:branch-0.14 Sep 2, 2025
15 checks passed
@jameslamb jameslamb deleted the drop-pynvml branch September 2, 2025 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants