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

Issue 122563003: Refactor calculation of "other regressions" output. (Closed)

Created:
6 years, 11 months ago by shatch
Modified:
6 years, 11 months ago
Reviewers:
tonyg
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Refactor calculation of "other regressions" output, calculate confidence in these local regressions, and only output if high confidence. Previously, we output every change if it was greater than some small %, which resulted in a lot of noise. Example output: Other regressions may have occurred: Depot Range Confidence chromium fc062604f35d88e607e36ae5f91d19d7cb2251a2..400664c4101576b48e30da67d57ecffc4d83fcb0 95% BUG=319898 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243246

Patch Set 1 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+77 lines, -70 lines) Patch
M tools/bisect-perf-regression.py View 5 chunks +77 lines, -70 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
shatch
6 years, 11 months ago (2014-01-06 20:03:59 UTC) #1
tonyg
lgtm Very nice!
6 years, 11 months ago (2014-01-06 20:16:07 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/simonhatch@chromium.org/122563003/50001
6 years, 11 months ago (2014-01-06 21:11:48 UTC) #3
commit-bot: I haz the power
6 years, 11 months ago (2014-01-07 10:47:16 UTC) #4
Message was sent while issue was closed.
Change committed as 243246

Powered by Google App Engine
This is Rietveld 408576698