Skip to content
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

Add GPU memory usage to dask_memusage plugin. #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jcfaracco
Copy link

Today, it is almost impossible to monitor GPU usage using dask_memusage. Only regular memory is possible.

With the proposed commits it is possible at least to compare the minimum GPU usage with the memory pick using GPUtil library.

The only problem with this approach is that we cannot track memory usage by each process by using nvidia-smi pmon -c 1 -s m for example. For some reason, this command does not work inside docker containers. It is possible to monitor only from the host system.

This commit just monitors GPU memory usage for each dask task using
GPUtil. This library is just a parser of nvidia-smi queries. There is no
way today to monitor GPU usage programatically using dask_memusage. Only
generating reports from Dask.

Signed-off-by: Julio Faracco <[email protected]>
In some cases, Dask client throws an error when the tests finish. If we
shutdown the client, the tests run smoothly.

Signed-off-by: Julio Faracco <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant