Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Support event statistics view in Theia #16

Open
1 of 2 tasks
tahini opened this issue May 25, 2020 · 2 comments
Open
1 of 2 tasks

Support event statistics view in Theia #16

tahini opened this issue May 25, 2020 · 2 comments
Labels
Theia UI frontend Trace Compass Involve changes in the main Trace Compass repo Trace Server Involve changes in the trace server itself (in the incubator source code)
Milestone
post-MVP

Comments

@tahini
Copy link
Contributor

tahini commented May 25, 2020 •
edited by MatthewKhouzam
Loading

This is the type of categorical view that can be used as base output

  • Make sure the data is provided through a data provider in Trace Compass
  • Support this data type with pie chart in Theia

eventStatistics
@tahini tahini added Trace Compass Involve changes in the main Trace Compass repo Trace Server Involve changes in the trace server itself (in the incubator source code) Theia UI frontend labels May 25, 2020
@tahini tahini added this to the First MVP milestone May 25, 2020
@ebugden ebugden modified the milestones: MVP, 1.0 Sep 30, 2021
@MatthewKhouzam
Copy link
Contributor

Pie charts remain, I would suggest they are blocked by D3's progress. @Rodrigoplp-work thoughts?

@Rodrigoplp-work
Copy link
Contributor

Exactly! With the structure we are already deploying in the y-axis plus the examples discussed in the "0003-D3-charts" ADR, this would fall perfectly in line for expanding our use of the library.

dleclairbb pushed a commit to dleclairbb/theia-trace-extension that referenced this issue Oct 25, 2023
@bhufmann
Use react-components for open traces and views and related updates
1287ba2 
- This change includes updates to propagate signals between webviews.
This will allow views to be updated when one view changes. For example,
when the user selects an open trace in the "Traces" view, the selection
is sent to the various react-components. The "Available Views" view
will use that signal as trigger to fetch the available views from
the remote server.

- Remove analysis and traces tree view from the extension. They are
replaced by the theia-trace-extension react components and
corresponding webviews.

- The classes AnalysisTree and TracesTree is still in the code base to
allow opening traces from the file system using the context sensitive
menu and "Open in Trace Viewer" command. This will have to be clean-up
in a later patch (+ using a cancelable progress monitor).

- The patch also includes a fix to make source-map work when debugging
the webview apps.

Needs eclipse-cdt-cloud#295

Fixes eclipse-cdt-cloud#7
Fixes eclipse-cdt-cloud#11
Fixes eclipse-cdt-cloud#16

Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
Theia UI frontend Trace Compass Involve changes in the main Trace Compass repo Trace Server Involve changes in the trace server itself (in the incubator source code)
Projects
None yet
Milestone
post-MVP
Development

No branches or pull requests
4 participants
@MatthewKhouzam @tahini @ebugden @Rodrigoplp-work