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

Issue 2707603002: [Predator] Generate all changelogs in regression ranges instead of only matched changelogs (Closed)

Created:
3 years, 10 months ago by Sharu Jiang
Modified:
3 years, 10 months ago
CC:
chromium-reviews, infra-reviews+infra_chromium.org, stgao, ymzhang1, inferno
Target Ref:
refs/heads/master
Project:
infra
Visibility:
Public.

Description

[Predator] Generate all changelogs in regression ranges instead of only matched changelogs Also clean up some scorer files and tests. BUG=693822 TBR=stgao@chromium.org Review-Url: https://codereview.chromium.org/2707603002 Committed: https://chromium.googlesource.com/infra/infra/+/ac0b35cd815c73ece94a5ac048d4e3b30122412e

Patch Set 1 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+147 lines, -1809 lines) Patch
D appengine/findit/crash/changelist_classifier.py View 1 chunk +0 lines, -314 lines 0 comments Download
M appengine/findit/crash/loglinear/changelist_classifier.py View 2 chunks +26 lines, -23 lines 0 comments Download
M appengine/findit/crash/loglinear/changelist_features/test/min_distance_test.py View 4 chunks +4 lines, -12 lines 0 comments Download
M appengine/findit/crash/loglinear/changelist_features/test/top_frame_index_test.py View 1 chunk +1 line, -1 line 0 comments Download
M appengine/findit/crash/loglinear/changelist_features/test/touch_crashed_file_meta_test.py View 1 chunk +0 lines, -3 lines 0 comments Download
M appengine/findit/crash/loglinear/changelist_features/test/touch_crashed_file_test.py View 1 chunk +1 line, -1 line 0 comments Download
M appengine/findit/crash/loglinear/changelist_features/touch_crashed_file_meta.py View 4 chunks +43 lines, -4 lines 0 comments Download
M appengine/findit/crash/loglinear/test/changelist_classifier_test.py View 3 chunks +54 lines, -84 lines 0 comments Download
D appengine/findit/crash/scorers/__init__.py View 1 chunk +0 lines, -3 lines 0 comments Download
D appengine/findit/crash/scorers/aggregated_scorer.py View 1 chunk +0 lines, -34 lines 0 comments Download
D appengine/findit/crash/scorers/aggregators.py View 1 chunk +0 lines, -107 lines 0 comments Download
D appengine/findit/crash/scorers/min_distance.py View 1 chunk +0 lines, -85 lines 0 comments Download
D appengine/findit/crash/scorers/scorer.py View 1 chunk +0 lines, -45 lines 0 comments Download
D appengine/findit/crash/scorers/test/__init__.py View 1 chunk +0 lines, -3 lines 0 comments Download
D appengine/findit/crash/scorers/test/aggregated_scorer_test.py View 1 chunk +0 lines, -64 lines 0 comments Download
D appengine/findit/crash/scorers/test/aggregators_test.py View 1 chunk +0 lines, -48 lines 0 comments Download
D appengine/findit/crash/scorers/test/min_distance_test.py View 1 chunk +0 lines, -93 lines 0 comments Download
D appengine/findit/crash/scorers/test/scorer_test_suite.py View 1 chunk +0 lines, -39 lines 0 comments Download
D appengine/findit/crash/scorers/test/top_frame_index_test.py View 1 chunk +0 lines, -34 lines 0 comments Download
D appengine/findit/crash/scorers/top_frame_index.py View 1 chunk +0 lines, -49 lines 0 comments Download
M appengine/findit/crash/stacktrace.py View 1 chunk +14 lines, -0 lines 0 comments Download
M appengine/findit/crash/suspect.py View 3 chunks +1 line, -135 lines 0 comments Download
D appengine/findit/crash/test/changelist_classifier_test.py View 1 chunk +0 lines, -517 lines 0 comments Download
M appengine/findit/crash/test/suspect_test.py View 3 chunks +3 lines, -111 lines 0 comments Download

Messages

Total messages: 18 (11 generated)
Sharu Jiang
PTAL.
3 years, 10 months ago (2017-02-18 01:56:04 UTC) #4
Sharu Jiang
ping :)
3 years, 10 months ago (2017-02-21 21:54:05 UTC) #6
Martin Barbella
LGTM. Sorry for the delay.
3 years, 10 months ago (2017-02-21 23:35:21 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2707603002/20001
3 years, 10 months ago (2017-02-22 02:17:54 UTC) #10
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
3 years, 10 months ago (2017-02-22 02:17:56 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2707603002/20001
3 years, 10 months ago (2017-02-22 02:32:24 UTC) #15
commit-bot: I haz the power
3 years, 10 months ago (2017-02-22 02:42:12 UTC) #18
Message was sent while issue was closed.
Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/ac0b35cd815c73ece94a5ac048d4e...

Powered by Google App Engine
This is Rietveld 408576698