flutter / flutter/devtools

"frame times may be negatively affected" is not quite the right message

Offen
#3,530 9 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
needs discussion P3 polish screen: performance
Vorherrschende Sprache
Dart
Sterne
1.7k
Forks
404
Ø Merge
6 T. 17 Std.
Gemergte PRs (30 T.)
18

Beschreibung

We have these checkboxes that add more instrumentation:
https://github.com/flutter/devtools/blob/master/packages/devtools_app/lib/src/performance/performance_screen.dart#L389

We have a message that says that "frame times may be negatively affected" when these are on, which is true. However, I'm not sure that fully represents the issue with these flags. People might see that and think, ok, I'll ignore the _numbers_, but the _relative_ sizes of work loads is still a guide as to what I should work on... which is wrong. It's possible for a widget to take 5ms with these flags off, and be the main problem that causes jank, but for these flags to cause other widgets to take 5ms as well while the problematic widget continues to only take 5ms, and now the source of the jank has been obscured.

The problem is that the frame times can be affected in _non-representative ways_.

Anyway I tried to submit a PR to fix this but I could not come up with wording that was accurate and comprehensible. So I'm filing this bug instead.

cc @InMatrix

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Untersuche packages/devtools_app/lib/src/performance/performance_screen.dart im Bereich der Instrumentierungs-Checkboxen und der bestehenden Meldung "Frame-Zeiten können negativ beeinflusst werden". Überarbeite die Formulierung so, dass Benutzer verstehen, dass Instrumentierung Frame-Workloads auf nicht repräsentative Weise verzerren kann, einschließlich relativer Vergleiche, und überprüfe anschließend die Meldung im Performance-Bildschirm.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
dart
Bereich
performance, tooling
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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