-
Notifications
You must be signed in to change notification settings - Fork 54
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
print_timer: show minima instead of avg, and switch off totals and/or %tot ? #160
Comments
More customization for the table makes sense. Perhaps one should be able to explicitly give all the columns one wants to see which would allow a larger spread of choices. |
In my own fork I have hacked the code to output this:
And honestly it looks pretty easy to do. I am not making a pull request because there are some things that can went wrong during development and the test suite did not catch them. |
Similar here. In my fork, I also computed the median by using OnlineStats.jl. |
I am enjoying your package, since it is simple to use (and one does not spend a long time waiting as with
@btime
which takes >1sec even if a short total is requested...).The usage
to(myfunc)(my,args)
makes the timing code particularly lightweight :)An obvious and simple feature request would be an option in
print_timer
to replace the "avg" column with "min" (this is like@btime
which reports the min). This is less sensitive to compilation on the first sample, or to interrupts.I imagine people would also like to switch off the "time" and "%tot" columsn, and just focus on the "min" or "avg". Maybe I didn't notice if there are options already?
Thanks! Alex
The text was updated successfully, but these errors were encountered: