-
Notifications
You must be signed in to change notification settings - Fork 158
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
Fix SLURM RAM monitoring #473
Conversation
e894337
to
e61a581
Compare
Tested on a SLURM environment : there is only too call per sessions and no more error message. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Benoît! Small comment about the notebook, not sure we want to merge it to master
I'll have a look next week 😊 |
There was an issue with |
This works as expected from codecarbon.external.hardware import RAM
ram = RAM(tracking_mode="process")
ram.slurm_memory_GB |
Thanks for your review, it seems to be deprecated since 2017 so it is safe to remove the support of the old name ! |
This accounts for the deprecated `$SLURM_JOBID`
e46021e
to
b4c39f7
Compare
Fix SLURM RAM monitoring :
Will close #447