eclipse-cdt-cloud / theia-trace-extension

Eclipse Theia trace viewer extension using the Trace Server Protocol (TSP), through the tsp-typescript-client. Also the home for reusable JavaScript libraries: traceviewer-base, traceviewer-react-components
MIT License
48 stars 60 forks source link

Redundant views shown for LTTng UST traces #699

Open bhufmann opened 2 years ago

bhufmann commented 2 years ago

With the support of data-tree and scatter charts views like Latency Statistics or Latency vs Time views are now available in the list of Available Views. The Trace Compass server of today provides call stack analysis from the Trace Compass mainline and the Trace Compass Incubator which leads to duplicate and redundant views:

From Trace Compass:

From Trace Compass Incubator:

The statistics views show pretty much same data, with some small differences. In some cases the symbol resolution is not working all the time for LTTng-UST CallStack (Incubator) - Latency Statistics.

The Flame Chart LTTng-UST CallStack (Incubator) also shows the Kernel Status if the the kernel trace is part of the trace set. This is not available in the mainline Flame Chart.

Here is picture of the available views for a LTTng UST trace: image

To avoid confusion for end-users (and new users), the list of available view needs to be cleaned-up to remove duplicates and redundant data. This is a Trace Compass server implementation issue an needs to be handled there and not in the front-end because the front-end is server-agnostic.

bhufmann commented 2 years ago

Relates to #296