Commit 8ec61a7e authored by ernstm@chromium.org's avatar ernstm@chromium.org

telemetry: make mostly_smooth in smoothness metric a float value.

The mostly_smooth metric doesn't show up on the dashboard,
because it is a boolean value (False/True). Switch to float (0.0/1.0).

R=dtu@chromium.org
BUG=

Review URL: https://codereview.chromium.org/54793006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232974 0039d316-1c4b-4281-b951-d872f2087c98
parent 6da14df5
...@@ -84,7 +84,7 @@ class SmoothnessMetric(Metric): ...@@ -84,7 +84,7 @@ class SmoothnessMetric(Metric):
jank = statistics.FrameDiscrepancy(self._stats.frame_timestamps) jank = statistics.FrameDiscrepancy(self._stats.frame_timestamps)
results.Add('jank', '', round(jank, 4)) results.Add('jank', '', round(jank, 4))
# Are we hitting 60 fps for 95 percent of all frames? (Boolean value) # Are we hitting 60 fps for 95 percent of all frames?
# We use 17ms as a slightly looser threshold, instead of 1000.0/60.0. # We use 17ms as a slightly looser threshold, instead of 1000.0/60.0.
results.Add('mostly_smooth', '', percentile_95 = statistics.Percentile(self._stats.frame_times, 95.0)
statistics.Percentile(self._stats.frame_times, 95.0) < 17.0) results.Add('mostly_smooth', '', 1.0 if percentile_95 < 17.0 else 0.0)
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment