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

Unified Diff: tools/testing/perf_testing/run_perf_tests.py

Issue 9724006: Fix typo on run_perf_tests.py. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/perf_testing/run_perf_tests.py
===================================================================
--- tools/testing/perf_testing/run_perf_tests.py (revision 5596)
+++ tools/testing/perf_testing/run_perf_tests.py (working copy)
@@ -195,7 +195,7 @@
for f in os.listdir(data):
files += [(os.path.getmtime(os.path.join(data, f)), f)]
files.sort()
- for f in files[-1000]:
+ for f in files[-1000:]:
shutil.copyfile(os.path.join(data, f[1]),
os.path.join(path, f[1]+'.txt'))
# Generate directory listing.
@@ -386,8 +386,9 @@
"""Generate a plot that shows the performance changes of the geomentric mean
of JS and frog benchmark performance over svn history."""
(title, y_axis, size_x, size_y, loc, filename) = \
- ('Geometric Mean of benchmark %s performance' % self.platform_type,
- 'Speed (bigger = better)', 16, 5, 'lower left', 'avg'+png_filename)
+ ('Geometric Mean of benchmark %s performance on %s ' % \
vsm 2012/03/16 22:56:18 Not sure you need the trailing \ at the end of the
+ (self.platform_type, utils.GuessOS()), 'Speed (bigger = better)', 16, 5,
vsm 2012/03/16 22:56:18 Extra whitespace at end of line?
+ 'lower left', 'avg'+png_filename)
clear_axis = True
for platform in self.platform_list:
self.style_and_save_perf_plot(title, y_axis, size_x, size_y, loc,
« 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