-
Notifications
You must be signed in to change notification settings - Fork 19
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
Pr add prover benchmark tool #790
Conversation
benchmarks/utils.nim
Outdated
for v in vals: | ||
elapsedStr &= ", " & v.formatFloat(format = ffDecimal, precision = 3) | ||
stdout.styledWriteLine( | ||
fgGreen, "CPU Time [", benchmarkName, "] ", "avg(", $nn, "): ", elapsedStr, " s" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we actually derive an average for the number of runs here? in fact, I would print this on it's own line at the end of the bench run.
Something like:
"CPU Time ["... "], ...s"
...
"Average for ${nn} runs is ...."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely, I was thinking about doing that. The current form is because I wanted to print out each run so I could do a simple linear regression on it. It's a pain to work with though in this form.
cleanup wording Signed-off-by: Dmitriy Ryajov <dryajov@gmail.com>
No description provided.