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

Prettier output #5

Closed
wants to merge 11 commits into from
Closed

Prettier output #5

wants to merge 11 commits into from

Conversation

simonLeary42
Copy link

This is all for slurm_gpustat:

  • rearranged outputs: up, accessible, per-user, available -> up, accessible, available, per-user
  • normalized the format of the outputs for count: gpu-name
  • put the per-user section into a PrettyTable so that data is aligned
  • sorted the available section because the other sections are sorted
  • changed gpus to GPU's
  • changed up to configured because it could mean down or drained
  • changed accessible to up because it could mean unavailable
  • made the divider an ALL_CAPS top-of-file variable
  • made printing slurm_str optional
  • made 'interactive' part of per-user optional

@simonLeary42
Copy link
Author

I see slurm_gpustat is a separate repo, I will take my PR elsewhere

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.

None yet

1 participant