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

positional parameters siganl repetition (a la vmstat) #25

Closed
wants to merge 1 commit into from

Conversation

asdkant
Copy link

@asdkant asdkant commented Sep 17, 2017

After raising the issue #24 I thought it'd be an easy thing to do, and did it. I tested it with both python 2 and 3.

@asdkant asdkant mentioned this pull request Sep 17, 2017
@wookayin
Copy link
Owner

Thanks for your contribution. I was considering to use not positional arguments, but optional (keyword) arguments such as --watch, and it remains controversial in my mind.

And, I feel it is much easier to see what is changing if we use something like watch -n1 gpustat, as top or htop command does, rather than simply appending the output.

@asdkant
Copy link
Author

asdkant commented Sep 17, 2017

Well, this IS a "(somethng)stat" command, so I just figured it'd be natural to follow the *stat convention

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

2 participants