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

Allow code heatmap to show time taken per line as well as execution count #58

Closed
mdfirman opened this issue Jul 13, 2016 · 6 comments
Closed

Comments

@mdfirman
Copy link

Thanks for a great tool! This visual profiler is what I was looking for in Python.

I was surprised though that the code heatmap only shows the execution count for the colour highlighting, and for the mouseovers. For me it is far more interesting to know how long each line took, not just how many times it is run. Of course this is also somewhat in the flame graph, but not in the same format.

It would be great to be able to select what the code heatmap shows, effectively allowing the user to choose between the columns of line_profiler. This could be in the command-line options, but ideally could be in a drop-down menu in the code heatmap screen. I believe matlab's profiler offers something similar.

Thanks

@nvdv
Copy link
Owner

nvdv commented Jul 14, 2016

Thanks for reporting!

@dlorch
Copy link

dlorch commented Feb 11, 2017

Upvote +1 for this feature request ;-)

@nvdv
Copy link
Owner

nvdv commented Mar 8, 2017

I've just pushed a few commits with initial version of this feature. Might be rough around the edges.
That's how code heatmap looked like:
exec_heatmap
That's how it looks now:
time_heatmap

@dlorch
Copy link

dlorch commented Mar 8, 2017

This looks very cool!

@nvdv
Copy link
Owner

nvdv commented Mar 9, 2017

Thanks.
This feature still requires some improvements and more testing, so I will post updates here.

@nvdv
Copy link
Owner

nvdv commented Apr 9, 2017

I've just uploaded this feature to PyPI. At present It tracks main thread only, since it does not require major changes to UI.

@nvdv nvdv closed this as completed Apr 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants