Skip to content

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.

@jooris-hadeler
Copy link
Author

It has been two years since I opened this Pull request. The last communication on this thread was that there is already someone who implemented this in a PR, yet after two years none of these have been merged. I would love to know if we can get this PR merged or if I should close this PR?

@sharkdp sharkdp reopened this Jan 25, 2025
@sharkdp
Copy link
Owner

sharkdp commented Jan 25, 2025

Let's please leave it open, I am still planning to do a hyperfine 2.0 release soon (I'm not giving any specific dates), and I am planning to integrate this as well.

Sorry that it takes so long.

@jooris-hadeler
Copy link
Author

Okay no worries, if there is anything you want changed I'm happy to integrate your feedback.

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