Skip to content

feat: add aggregated and bottom-up call tree views#755

Merged
lcottercertinia merged 19 commits intocertinia:mainfrom
lukecotter:feat-calltree-bottom-up-and-others
Apr 29, 2026
Merged

feat: add aggregated and bottom-up call tree views#755
lcottercertinia merged 19 commits intocertinia:mainfrom
lukecotter:feat-calltree-bottom-up-and-others

Conversation

@lukecotter
Copy link
Copy Markdown
Contributor

@lukecotter lukecotter commented Apr 29, 2026

📝 PR Overview

Adds aggregated and bottom-up call tree views, aligns analysis table behavior with the new call tree tables, and updates related docs so log analysis workflows are consistent and easier to use.

🛠️ Changes made

  • Add aggregated, bottom-up, and time-order call tree table implementations with shared aggregation utilities
  • Refactor call tree and analysis views to share table behavior, filtering, grouping, and persistence logic
  • Add tests for aggregation, row grouping, min/max filtering, and bottom calculation behavior
  • Update README, changelog, and docs for the new analysis and call tree functionality

🧩 Type of change (check all applicable)

  • 🐛 Bug fix - something not working as expected
  • ✨ New feature – adds new functionality
  • ♻️ Refactor - internal changes with no user impact
  • ⚡ Performance Improvement
  • 📝 Documentation - README or documentation site changes
  • 🔧 Chore - dev tooling, CI, config
  • 💥 Breaking change

📷 Screenshots / gifs / video [optional]

image image

🔗 Related Issues

fixes #
resolves #333
closes #
related #

✅ Tests added?

  • 👍 yes
  • 🙅 no, not needed
  • 🙋 no, I need help

📚 Docs updated?

  • 🔖 README.md
  • 🔖 CHANGELOG.md
  • 📖 help site
  • 🙅 not needed

Anything else we need to know? [optional]

The branch also includes internal table and filter refactors that support the new call tree views and keep analysis behavior aligned.

- Split CalltreeView into TimeOrderTable, AggregatedTable, BottomUpTable
- Add Aggregation, MergeAdjacent, BottomCalcs, Multiset utilities + tests
- Split MinMaxFilter into core (non-recursive) and minMaxTreeFilter (recursive, dotted-path safe)
   — fixes min/max on the top-down view
- BottomUp uses the non-recursive filter (children are callers, not contained calls)
- makeSumSelfTimeAllVisible now walks filtered tree children, so the self-time bottom calc respects filters across collapsed branches
…creation to return table and promise for built state
@lcottercertinia lcottercertinia changed the title Add aggregated and bottom-up call tree views feat: add aggregated and bottom-up call tree views Apr 29, 2026
@lcottercertinia lcottercertinia merged commit 0ff08cf into certinia:main Apr 29, 2026
5 checks passed
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.

feat: call tree - time order, aggregate, bottom up views

2 participants