Skip to content

Conversation

@bhufmann
Copy link
Contributor

@bhufmann bhufmann commented Jun 10, 2025

What it does

  • tmf: Add GANTT_CHART data provider type
  • Expose FlameGraphDataProvider using new data provider type. Implement getDescriptor method for FlameGraphDataProviderFactory so that it's visible when running the trace server.

How to test

Query available data providers using Python client.

Try the preview of UI implementation: eclipse-cdt-cloud/theia-trace-extension#1205

Follow-ups

N/A

Review checklist

  • As an author, I have thoroughly tested my changes and carefully followed the instructions in this template

@bhufmann bhufmann force-pushed the flame-graph-dp branch 2 times, most recently from f282a69 to 9a51851 Compare June 10, 2025 18:58
@bhufmann bhufmann changed the title Do not merge! Test flame graph data provider tmf: Add GANTT_CHART data provider type and use it for flame graphs Jul 11, 2025
MatthewKhouzam
MatthewKhouzam previously approved these changes Jul 11, 2025
Copy link
Contributor

@MatthewKhouzam MatthewKhouzam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is simple and clear. I like it!

[Added] GANTT_CHART data provider type and use it for flame graphs

Signed-off-by: Bernd Hufmann <[email protected]>
Copy link
Contributor

@MatthewKhouzam MatthewKhouzam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@bhufmann bhufmann merged commit d487cf2 into eclipse-tracecompass:master Jul 11, 2025
4 checks passed
@bhufmann bhufmann deleted the flame-graph-dp branch July 11, 2025 15:24
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