Migrate pynvml to cuda.core.system#146
Conversation
f6a44df to
d88499b
Compare
|
I was trying to install this in Colab for example to give it a try and see if it works as expected, and I ran into an installation ERROR It seems to be updating cuda bindings to the latest, which is to aggressive for most CSP. I understand that the numba cuda version in colab is super old. So we can report that back and see if we can get them to upgrade. All that being said, we can run the script. At the moment we are running the script to check the environment, but it seems like with this changes installing the rapids-cli is modifying that environment. I'd like to know what @jacobtomlinson, @jayavenkatesh19 and @mmccarty think here. |
Following up on #146 (comment), usage of `cuda-core` has a specific pin on `cuda-bindings` version which has different versions for CUDA 12 and CUDA 13. While `cuda-bindings` itself is backward compatible, this pin is causing issues on other environments where tools like `pytorch` have their own versioning requirements for `cuda-bindings`. By switching back to using `pynvml` for GPU information, we can ensure that `rapids-cli` is supported on diverse environments without the need for specialized wheels --------- Signed-off-by: Jaya Venkatesh <jjayabaskar@nvidia.com>
Can you move to cuda-python 12.9.6? That should hopefully be enough to satisfy the dependencies here. Unfortunately, 12.9.4 doesn't have the NVML functionality at all. |
@mdboom The problem is that in the case of rapids-cli, we use this library, among other things to run It's also bit problematic for us, that installing rapids-cli, will mean to upgrade a package in the environment, that we are trying to collect information on. We talked about this with the team, and decided that we will hold of on the migration until the cc: @jayavenkatesh19 |
I think "more mature" is an ambiguous term. This is ultimately yet another packaging (aka dependency hell) problem and by "more mature" we don't mean that there is anything actionable for |
Migrates from pynvml.py to the new Cython/cybind-based cuda.core.system API.