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

Side by Side Diff: tools/bisect-perf-regression.py

Issue 499653002: Use correct string format to get bisect confidence. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Performance Test Bisect Tool 6 """Performance Test Bisect Tool
7 7
8 This script bisects a series of changelists using binary search. It starts at 8 This script bisects a series of changelists using binary search. It starts at
9 a bad revision where a performance metric has regressed, and asks for a last 9 a bad revision where a performance metric has regressed, and asks for a last
10 known-good revision. It will then binary search across this revision range by 10 known-good revision. It will then binary search across this revision range by
(...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 # The perf dashboard looks for a string like "Estimated Confidence: 95%" 192 # The perf dashboard looks for a string like "Estimated Confidence: 95%"
193 # to decide whether or not to cc the author(s). If you change this, please 193 # to decide whether or not to cc the author(s). If you change this, please
194 # update the perf dashboard as well. 194 # update the perf dashboard as well.
195 RESULTS_BANNER = """ 195 RESULTS_BANNER = """
196 ===== BISECT JOB RESULTS ===== 196 ===== BISECT JOB RESULTS =====
197 Status: %(status)s 197 Status: %(status)s
198 198
199 Test Command: %(command)s 199 Test Command: %(command)s
200 Test Metric: %(metrics)s 200 Test Metric: %(metrics)s
201 Relative Change: %(change)s 201 Relative Change: %(change)s
202 Estimated Confidence: %(confidence)d%%""" 202 Estimated Confidence: %(confidence).02f%%"""
203 203
204 # The perf dashboard specifically looks for the string 204 # The perf dashboard specifically looks for the string
205 # "Author : " to parse out who to cc on a bug. If you change the 205 # "Author : " to parse out who to cc on a bug. If you change the
206 # formatting here, please update the perf dashboard as well. 206 # formatting here, please update the perf dashboard as well.
207 RESULTS_REVISION_INFO = """ 207 RESULTS_REVISION_INFO = """
208 ===== SUSPECTED CL(s) ===== 208 ===== SUSPECTED CL(s) =====
209 Subject : %(subject)s 209 Subject : %(subject)s
210 Author : %(author)s%(email_info)s%(commit_info)s 210 Author : %(author)s%(email_info)s%(commit_info)s
211 Commit : %(cl)s 211 Commit : %(cl)s
212 Date : %(cl_date)s""" 212 Date : %(cl_date)s"""
(...skipping 3143 matching lines...) Expand 10 before | Expand all | Expand 10 after
3356 # bugs. If you change this, please update the perf dashboard as well. 3356 # bugs. If you change this, please update the perf dashboard as well.
3357 bisect_utils.OutputAnnotationStepStart('Results') 3357 bisect_utils.OutputAnnotationStepStart('Results')
3358 print 'Error: %s' % e.message 3358 print 'Error: %s' % e.message
3359 if opts.output_buildbot_annotations: 3359 if opts.output_buildbot_annotations:
3360 bisect_utils.OutputAnnotationStepClosed() 3360 bisect_utils.OutputAnnotationStepClosed()
3361 return 1 3361 return 1
3362 3362
3363 3363
3364 if __name__ == '__main__': 3364 if __name__ == '__main__':
3365 sys.exit(main()) 3365 sys.exit(main())
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698