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

Unified Diff: scripts/slave/recipe_modules/bisect_tester/perf_test.py

Issue 1610203003: Iteratively increase sample size for good/bad classification. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Rebasing Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « scripts/slave/recipe_modules/bisect_tester/api.py ('k') | scripts/slave/recipes/bisect.expected/basic.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/bisect_tester/perf_test.py
diff --git a/scripts/slave/recipe_modules/bisect_tester/perf_test.py b/scripts/slave/recipe_modules/bisect_tester/perf_test.py
index adaf6f3a38c63996c76a8768852630d7ca0cc1e9..1b0c5b41fd2651e2b41ccd64010e6251880c49af 100644
--- a/scripts/slave/recipe_modules/bisect_tester/perf_test.py
+++ b/scripts/slave/recipe_modules/bisect_tester/perf_test.py
@@ -93,7 +93,8 @@ def run_perf_test(api, test_config, **kwargs):
start_time = time.time()
for i in range(repeat_cnt):
elapsed_minutes = (time.time() - start_time) / 60.0
- if elapsed_minutes >= limit: # pragma: no cover
+ # A limit of 0 means 'no timeout set'.
+ if limit and elapsed_minutes >= limit: # pragma: no cover
break
if is_telemetry:
if i == 0 and kwargs.get('reset_on_first_run'):
« no previous file with comments | « scripts/slave/recipe_modules/bisect_tester/api.py ('k') | scripts/slave/recipes/bisect.expected/basic.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698