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

Add -E and --show-elapsed flag to show the elapsed time. #706

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jooris-hadeler
Copy link

Changes

  • add the -E and --show-elapsed flags to optionally enable a elapsed time counter

Origin

Recently when I was benchmarking software for my university course, I found that hyperfine gives you very little information about how long the current run has been running. This is very unfortunate because you can't tell if your program has possibly deadlocked. This pull request adds the -E and --show-elapsed flags which will show you for how long the current run is running.

@sharkdp
Copy link
Owner

sharkdp commented Dec 1, 2023

Thank you. Did you see #416 and #581? How does this compare to previous approaches? Also, do you think we should make this enabled by default?

@jooris-hadeler
Copy link
Author

I must say I completely forgot to check if someone had already proposed this. Compared to #416 it is quite similar, although in comparison to #581 this would always show the time for the current run, regardless of it being the initial or a followup run. I think this feature is only really necessary in some specific cases (e.g. the mentioned long runs where it might crash), I think it would be wise to hide this behind a flag.

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.

2 participants