nicolargo / nvidia-ml-py3 Goto Github PK

First up, thanks for this extremely useful Lib.

I was wondering if it would be possible to push the latest version with the check into System32 for Windows users to PyPi?

It seems that Windows is increasingly installing Nvidia drivers this way, and it would be extremely useful to be able to pull this functionality in with Pip.

Many thanks, and keep up the good work!

I apologize for mis-using this github issue tracker @nicolargo . I wanted to have a public discussion on GitHub about Pythonic GPU diagnostic libraries and yours was the most relevant issue tracker that I found. I'm happy to move conversation elsewhere if desired. Also, thank you for porting this library to current Python.

That being said, this library has lots of functionality, but isn't particularly friendly for a typical Python user. It appears to be a faithful port of a C library both for good and bad.

Is there value to designing a new library that wrapped around this one and provided the same diagnostics but in a different way? If so, what thoughts do people have about API for that library?

cc @sklam @seibert @ogrisel @lesteve @beberg (please also ping others)

Thank you for providing this library.

I'm posting this in hopes that it will help someone who may have the same problem as I did.
When trying to print a class inherited from _PrintableStructure such as c_nvmlProcessInfo_t() using print(c_nvmlProcessInfo_t()), the following AttributeError occurred.

AttributeError: module 'string' has no attribute 'join'

This is because string.join() has been removed in python3.
The problem was solved by changing the value returned from __str__ of the _PrintableStructure class in pynvml.py as follows.

Original:
return self.__class__.__name__ + "(" + string.join(result, ", ") + ")"

Modified:
return self.__class__.__name__ + "(" + ", ".join(result) + ")"