New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 849321 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Oct 10
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Build-Toolchain



Sign in to add a comment

Add speedometer2 to toolchain perf tests

Project Member Reported by rahulchaudhry@chromium.org, Jun 4 2018

Issue description

Keep original speedometer in there as well for now.
 
Owner: yunlian@chromium.org
https://chromium-review.googlesource.com/1082954 adds speedometer2 to crosperf/experiment_factory.py

Still to do:
- Once we have json from a few runs, update _estimated_stddev (in crosperf/benchmark.py).
- Update the json to columnio translation program (crostc_perf).
- Add to the performance dashboards.

Assigning to the new sheriff for remaining tasks.
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 5 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/toolchain-utils/+/4358aa80c7e8fcd0709d7760a52855f2a8ecd24d

commit 4358aa80c7e8fcd0709d7760a52855f2a8ecd24d
Author: Yunlian Jiang <yunlian@google.com>
Date: Tue Jun 05 09:15:32 2018

add speedometer2 result format

This adds the speedometer2 result format to
default-telemetry-results.json.

BUG= chromium:849321 
TEST=run crosperf and it showed the result for speedometer2.

Change-Id: I8b824b6c1739a032bb15a706cee96d1f45c444ab
Reviewed-on: https://chromium-review.googlesource.com/1086292
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>

[modify] https://crrev.com/4358aa80c7e8fcd0709d7760a52855f2a8ecd24d/crosperf/default-telemetry-results.json

Status: Assigned (was: Untriaged)
This bug has an owner, thus, it's been triaged. Changing status to "assigned".
Status: Verified (was: Assigned)

Sign in to add a comment