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

Butterfly view #162

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Butterfly view #162

wants to merge 7 commits into from

Conversation

bric3
Copy link
Owner

@bric3 bric3 commented Apr 7, 2023

Currently work in progress to produce a butterfly (also called a sandwich view).

Screenshot 2023-04-12 at 14 11 23

  • Produce an optimized and coherent butterfly view component, currently the view is based on two FlamegraphView
  • Fix the predecessors (callers), currently the weight and cumulative Weight are at odd with the converter's expectation.
    Now the weight and cumulativeWeight have a different meaning for predecessors, respectively the weight is the impact on the callers in the current backtrace, and cumulativeWeigth is the total amount of the predecessor in this profile (in the selected events)
  • Allow to enter this view dynamically, currently the filter is hardcoded for specific methods found in a specific profile.
  • Hide root node
  • Predecessors should be positioned at the bottom (might be fixed by not reusing FlamegraphVew)

Fix #108

@github-actions
Copy link

❌ Java CI with Gradle failed:

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.

Alternate visualizations for callers / callees trees (sandwich or butterfly)
1 participant