Repository navigation
feat(dashboards): Add trace metrics variants of the web vitals dashboards - #128924
edwardgou-sentry merged 27 commits into
Conversation
Sentry Snapshot Testing
|
📊 Type Coverage Diff
🔍 3 new type safety issues introducedNon-null assertions (
Type assertions (
This is informational only and does not block the PR. |
fb8daac to
ebd2c3e
Compare
…ware helper Lets a prebuilt config be swapped for a variant behind an org feature without touching every consumer.
…ards Served behind dashboards-web-vitals-trace-metrics. Samples and interactions stay on spans.
… of score formulas The trace metrics dataset has no performance_score or count_scores, so use avg and count. The pages table stays on spans.
ebd2c3e to
3aaea66
Compare
Drop the series label rework and extra tests the variants do not need.
…ards Served behind dashboards-web-vitals-trace-metrics. Samples and interactions stay on spans.
… of score formulas The trace metrics dataset has no performance_score or count_scores, so use avg and count. The pages table stays on spans.
Drop the series label rework and extra tests the variants do not need.
2466842 to
c397de8
Compare
…cs-config' of github.com:getsentry/sentry into edwardgou-sentry/feat/dashboards-web-vitals-trace-metrics-config
…try/feat/dashboards-web-vitals-trace-metrics-config
Drops the local unit widening now that DataUnit knows about fractions.
…cs dashboards wait on
| const isScoreBreakdown = isWebVitalScoreBreakdownChart(widget.queries[queryIndex]!); | ||
| return transformTimeSeriesResponse( | ||
| isScoreBreakdown | ||
| ? transformPerformanceScoreBreakdownTimeSeries(data, webVitalScoreField) |
There was a problem hiding this comment.
This logic is copied over from the spans version path.
Applies a weight to each web vital score series. The end result is that the "area" of all the series together form the total score (maxed out at 100).
…ics dashboards Matches the spans dashboards. It fills in once Relay copies geo attributes onto the metrics.
| import {ModuleName} from 'sentry/views/insights/types'; | ||
|
|
||
| // TODO: Move these to trace metrics once the metrics can back them. | ||
| const SPANS_WIDGET_IDS = new Set([ |
There was a problem hiding this comment.
TODO: several charts still use the spans dataset for now until attributes are more complete on metrics and necessary score functions are added to metrics dataset. Those blockers are being worked on in parallel.
|
@cursor review |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 9f1f71c. Configure here.
…entry/feat/dashboards-web-vitals-trace-metrics-config # Conflicts: # static/app/views/dashboards/datasetConfig/utils/traceMetricsResponseTransforms.ts

Adds a trace metrics version of the Web Vitals prebuilt dashboards. These dashboards are meant to replicate the functionality of the existing spans backed Web Vitals dashboards, but with trace metrics data. This is a work in progress and some of the widgets defined here still remain on spans while we work on unblocking in metrics.
This version is behind feature flag and won't be shown to users yet.
Scores come from
browser.web_vital.<vital>.scoreratio metrics emitted from Relay.Leverages
FractionUnitfor the score metrics from the stacked pr (#129337).