-
Notifications
You must be signed in to change notification settings - Fork 65
Adding gpu metrics collection command #498
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
base: main
Are you sure you want to change the base?
Conversation
SchneiderCode
left a comment
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.
Looks great! I added some suggestions for using -p and including the JobID in the sacct output.
Co-authored-by: Michael Schneider <m.schneider.programmer@gmail.com>
Co-authored-by: Michael Schneider <m.schneider.programmer@gmail.com>
Co-authored-by: Michael Schneider <m.schneider.programmer@gmail.com>
Co-authored-by: Michael Schneider <m.schneider.programmer@gmail.com>
Co-authored-by: Michael Schneider <m.schneider.programmer@gmail.com>
Co-authored-by: Michael Schneider <m.schneider.programmer@gmail.com>
SchneiderCode
left a comment
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.
Looks great! Only thing is we may need to wait until slurm tools is auto-loaded when users login. Or we may just need to include the "module load slurmtools" line that was deleted.
No description provided.