eclipse-cdt-cloud / eclipse-cdt-cloud/trace-server-protocol

Consolidate input query parameters across views for time ranges

Offen
#131 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
26
Forks
21
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

### Bug Description:

Consolidate input query parameters across views for time ranges.

Any time-based views uses the `requested_timerange` as input parameter to query the data for a give time range. Other views which don't have the trace time as x-axis don't always use `requested_timerange` for that.

For example, the GANTT_CHART data provider uses the requested_timerange to specify the start duration and end duration if the x-axis is duration. If the x-axis would be other data, e.g. bytes this input parameter would make even less sense. For the GANTT_CHART another query parameter exists in the Trace Compass to specify the time range to apply (see [here](https://github.com/eclipse-tracecompass/org.eclipse.tracecompass/blob/d487cf2e11795a2680d3333677f443e98724bcb9/analysis/org.eclipse.tracecompass.analysis.profiling.core/src/org/eclipse/tracecompass/internal/analysis/profiling/core/flamegraph/FlameGraphDataProvider.java#L506).

Another example are the Latency views, where it's possible to add the query parameter `requested_times` to specify start time (value at index 0) and end time (value at index 1). Additionally it is required to add the filter flag `isFiltered` with value true in the query parameters. Adding this will add total statistics and the time range based statistics in one query.

A third example is the generic xy chart implementation under development (see [here](https://github.com/eclipse-tracecompass/org.eclipse.tracecompass/pull/291#pullrequestreview-3055341970)) where the parameter `requested_timerange` is used for querying the data provider for specific time range.

This ticket is to align the input parameters and behaviour across data providers.

### Additional Information
N/A

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie damit, die Behandlung von requested_timerange und requested_times im GANTT_CHART-Provider, in FlameGraphDataProvider.java, in den Latency-Ansichten und in der in PR #291 referenzierten Arbeit an generischen XY-Charts zu vergleichen. Für das Issue müssen ein abgestimmter Vertrag für Eingabeparameter und ein konsistentes Verhalten über alle Datenprovider hinweg festgelegt werden; der Abschluss sollte für jede genannte Ansicht anhand dieses Vertrags überprüft werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
api
Issue-Typ
Bug
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.