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

Adding milliseconds #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

galano25
Copy link

@galano25 galano25 commented Mar 1, 2019

ttyclock -s -m will show with milliseconds

ttyclock -s -m will show with milliseconds
Copy link
Collaborator

@anarcat anarcat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this work at all? how fast is the refresh rate in ncurses anyways? can we really get ~10ms updates that do something here? it's not like humans can distinguish one milisecond to the next either. 10ms, maybe, but even that's a stretch...

not convinced this is worth it...

Copy link
Collaborator

@anarcat anarcat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, and i think you forgot to update the manpage. :)

@mheriyanto
Copy link

Is there any problem here?

@dashishmeister
Copy link

https://stackoverflow.com/questions/3808626/ncurses-refresh this seems pretty helpful, ngl I would love milliseconds for maximum hackerman vibes lol

@anarcat
Copy link
Collaborator

anarcat commented Aug 23, 2021

Is there any problem here?

yes, i have unanswered questions, see above. terminal latency is typically in the range of 10ms and up, and that's on a good day. i strongly doubt that we can show milisecond resolution, let alone have humans actually tell them apart.

chronometers typically only show two significant digits after the second, and the second one flashes by so fast that you can rarely tell it apart.

i'd say keep that to tenth of a second, and even then... i'm not sure i see the point.

@DrewSBAI
Copy link

DrewSBAI commented May 3, 2024

Very useful branch that should be merged into master. I used it to test the latency on a camera by recording the terminal and the streamed video.

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.

5 participants