Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(99)

Issue 847273002: Averaging nested result lists instead of flattening them. (Closed)

Created:
5 years, 11 months ago by RobertoCN
Modified:
5 years, 11 months ago
Reviewers:
prasadv, qyearsley
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Averaging nested result lists instead of flattening them. Also, removing special case where certain metrics where summed instead of averaged since they are now handled in the dashboard by averaging, no longer making them special cases. BUG=447588 Committed: https://crrev.com/658fcf7f1e37ac413ffbfa91c6e9c6e7acf10200 Cr-Commit-Position: refs/heads/master@{#311377}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -14 lines) Patch
M tools/auto_bisect/bisect_perf_regression.py View 2 chunks +2 lines, -14 lines 0 comments Download

Messages

Total messages: 8 (2 generated)
RobertoCN
How does this look?
5 years, 11 months ago (2015-01-13 23:35:58 UTC) #2
qyearsley
On 2015/01/13 23:35:58, robertocn wrote: > How does this look? LGTM -- I would add ...
5 years, 11 months ago (2015-01-13 23:57:03 UTC) #3
prasadv
lgtm
5 years, 11 months ago (2015-01-13 23:58:20 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/847273002/1
5 years, 11 months ago (2015-01-14 00:00:48 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 11 months ago (2015-01-14 00:50:35 UTC) #7
commit-bot: I haz the power
5 years, 11 months ago (2015-01-14 00:51:45 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/658fcf7f1e37ac413ffbfa91c6e9c6e7acf10200
Cr-Commit-Position: refs/heads/master@{#311377}

Powered by Google App Engine
This is Rietveld 408576698