scalene icon indicating copy to clipboard operation
scalene copied to clipboard

Could this package work with newer versions of nvidia-ml-py?

Open DManowitz opened this issue 2 years ago • 5 comments
trafficstars

I see from looking at previous issues that there was an issue with certain versions of nvidia-ml-py causing a problem because of a use of a print statement instead of a function. However, has anyone checked to see if newer versions of nvidia-ml-py still have this problem?

The reason I ask is that it looks like the nvida-ml-py package is being maintained more regularly than pynvml at this point (https://pypi.org/project/nvidia-ml-py/#history vs https://pypi.org/project/pynvml/#history), and I'm trying to set up an environment primarily using conda, which explicitly blocks installation of both pynvml and nvidia-ml-py (whereas I believe pip would just have the later install clobber the prior install).

DManowitz avatar Feb 08 '23 03:02 DManowitz

Thanks for letting us know; it had for long time seemed like nvidia-ml-py was in effect abandonware. IIRC the API is the same, but I have no idea if they have fixed the previous issues.

emeryberger avatar Feb 10 '23 14:02 emeryberger

I just tested this and it appears to work fine. Can you verify on your end? (Just uninstall pynvml)

emeryberger avatar Feb 10 '23 19:02 emeryberger

Another impact of depending on pynvml instead of nvidia-ml-py is that the latest versions of scalene cant be installed together with ray because ray depends on gpustat that depends on nvidia-ml-py that cant be installed together with pynvml

For example, creating a simple environment using conda create -n test_env python=3.9.18 ray-default=2.8.1 scalene=1.5.31.1 fails.

yonil7 avatar Dec 18 '23 08:12 yonil7

Actually, at the moment, pynvml has not been updated in a while, but nvidia-ml-py is getting regular updates.

DManowitz avatar Dec 18 '23 15:12 DManowitz

Yes, it seems that its better to depend on nvidia-ml-py instead of pynvml

yonil7 avatar Dec 18 '23 15:12 yonil7

@emeryberger Any thoughts about reverting this package to use nvidia-ml-py at this point? As a couple of us have mentioned, it seems to be getting updated much more regularly than pynvml as of now.

DManowitz avatar Jun 10 '24 15:06 DManowitz

As of the latest release (1.5.42), Scalene is back to using nvidia-ml-py.

emeryberger avatar Jun 16 '24 19:06 emeryberger

@emeryberger I just noticed that both nvidia-ml-py and nvidia-ml-py3 are specified as dependencies now. Why is that? From PyPi, it looks like nvidia-ml-py3 only had a single release 7 years ago. It seems that only nvidia-ml-py should be a dependency.

DManowitz avatar Jul 03 '24 18:07 DManowitz

@emeryberger Sorry to bother you, but could you release 1.5.42.3 (ideally to both PyPi and conda-forge) without the nvidia-ml-py3 dependency? Thanks.

DManowitz avatar Jul 13 '24 21:07 DManowitz