New issue
Advanced search Search tips

Issue 788841 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

telemetry: Include raw input-event latency data in the report

Project Member Reported by sadrul@chromium.org, Nov 27 2017

Issue description

The raw latency data includes some additional information that we
miss from reporting just the mean (e.g. the max, the variance etc.).
So it'd be useful to include the raw latency info in the report.

We can probably remove the mean_ and the _discrepancy metrics (since the raw data includes the mean, variance etc.). (although other tools/scripts may depend on those values existing right now)
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/catapult/+/6c4c2362a367c28ef6799012a8d06811b65b4a75

commit 6c4c2362a367c28ef6799012a8d06811b65b4a75
Author: Sadrul Habib Chowdhury <sadrul@chromium.org>
Date: Fri Dec 01 01:28:25 2017

Include the raw event latency data in the report.

The raw latency data includes some additional information that we
miss from reporting just the mean (e.g. the max, the variance etc.).
So include the raw latency info in the report.

BUG= chromium:788841 

Change-Id: I2a4b1741eb9667303eb31455847febd7fdcc47f4
Reviewed-on: https://chromium-review.googlesource.com/791273
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>

[modify] https://crrev.com/6c4c2362a367c28ef6799012a8d06811b65b4a75/telemetry/telemetry/web_perf/metrics/smoothness_unittest.py
[modify] https://crrev.com/6c4c2362a367c28ef6799012a8d06811b65b4a75/telemetry/telemetry/web_perf/metrics/smoothness.py

Project Member

Comment 2 by bugdroid1@chromium.org, Dec 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/18791a02a750de0c27b5adceafcc1345a0ea34e2

commit 18791a02a750de0c27b5adceafcc1345a0ea34e2
Author: catapult-deps-roller@chromium.org <catapult-deps-roller@chromium.org>
Date: Fri Dec 01 04:18:11 2017

Roll src/third_party/catapult/ 280aff07f..6c4c2362a (2 commits)

https://chromium.googlesource.com/catapult.git/+log/280aff07f5fc..6c4c2362a367

$ git log 280aff07f..6c4c2362a --date=short --no-merges --format='%ad %ae %s'
2017-11-28 sadrul Include the raw event latency data in the report.
2017-11-30 simonhatch Histograms - Fix deduplicate

Created with:
  roll-dep src/third_party/catapult
BUG= 788841 


The AutoRoll server is located here: https://catapult-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: Ibd00df50d2332b7d22a9891c541c50bc6f69b5b8
Reviewed-on: https://chromium-review.googlesource.com/802541
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520852}
[modify] https://crrev.com/18791a02a750de0c27b5adceafcc1345a0ea34e2/DEPS

Comment 3 by sadrul@chromium.org, Dec 12 2017

Status: Fixed (was: Started)

Sign in to add a comment