-
Notifications
You must be signed in to change notification settings - Fork 370
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
Summarize Profile View #26
Comments
Great idea! This should probably even be the default view. The current view very much pushes all the information it has into your face; having an overview first and then allowing the user to drill down as they want sounds much nicer to me. |
Yeah, that makes sense. Less is probably more when starting with this fire-hose of data. |
Some initial work is here, I've got the original cleopatra format summarizing the data, but need to get it working with the new format. https://github.com/gregtatum/cleopatra/blob/categories/src/summarize-profile.js |
I have the view built out now, but we can't switch between views yet. I'm going to maintain this separate for a bit and try out some things. |
A way to extend categories in the interface would be nice, as you might want to go from a higher level view to more details in one category (js -> event-handler vs request-animation-frame, js -> baseline vs ion jitted). |
I was talking with sfink about his needs, and he was wanting a top summary view of a profile that shows where time was being executed so that it would be really easy to point a finger at performance problems to specific parties.
This gist contains his existing Python script for performing this analysis. He admitted the categorization is a bit hacky, so maybe this could tie into Cleopatra's own categorization system.
The text was updated successfully, but these errors were encountered: