Skip to content

ui: Flamegraph expansion handles grouped by label nodes#5591

Merged
manojVivek merged 3 commits into
mainfrom
flamegraph-expansion-bug-fix
Apr 21, 2025
Merged

ui: Flamegraph expansion handles grouped by label nodes#5591
manojVivek merged 3 commits into
mainfrom
flamegraph-expansion-bug-fix

Conversation

@manojVivek
Copy link
Copy Markdown
Contributor

@manojVivek manojVivek commented Apr 21, 2025

The latest flamegraph expansion logic was broken when the flamegraph is grouped by labels and this fix handles that scenario.

@manojVivek manojVivek requested a review from a team as a code owner April 21, 2025 07:22
@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented Apr 21, 2025

🤖 Meticulous evaluated 50 user flows and took 334 visual snapshots. Meticulous has not yet run on 5271033 of the main branch and so there was nothing to compare against.

If you recently setup Meticulous, this is expected. Meticulous will start reporting comparisons for new pull requests after the next commit to the main branch.

Last updated for commit 656a49e. This comment will update as new commits are pushed.

@manojVivek manojVivek merged commit fb1d6a3 into main Apr 21, 2025
@manojVivek manojVivek deleted the flamegraph-expansion-bug-fix branch April 21, 2025 10:00
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.

2 participants