Index: ct/py/csv_comparer.py |
diff --git a/ct/py/csv_comparer.py b/ct/py/csv_comparer.py |
index d1f7868949cf9b3944e5d3c5bdffb23dfbe6027c..733b8a98e05f85e50f123b13afd7e2854cd4d08a 100644 |
--- a/ct/py/csv_comparer.py |
+++ b/ct/py/csv_comparer.py |
@@ -63,7 +63,7 @@ class CsvComparer(object): |
"""Class that compares two telemetry CSV files and outputs HTML results.""" |
def __init__(self, csv_file1, csv_file2, output_html_dir, requester_email, |
- chromium_patch_link, skia_patch_link, |
+ chromium_patch_link, skia_patch_link, benchmark_patch_link, |
variance_threshold, absolute_url, min_pages_in_each_field, |
discard_outliers, raw_csv_nopatch, raw_csv_withpatch, |
num_repeated, target_platform, crashed_instances, |
@@ -77,6 +77,7 @@ class CsvComparer(object): |
self._requester_email = requester_email |
self._chromium_patch_link = chromium_patch_link |
self._skia_patch_link = skia_patch_link |
+ self._benchmark_patch_link = benchmark_patch_link |
self._variance_threshold = float(variance_threshold) |
self._absolute_url = absolute_url |
self._min_pages_in_each_field = min_pages_in_each_field |
@@ -296,6 +297,7 @@ class CsvComparer(object): |
'requester_email': self._requester_email, |
'chromium_patch_link': self._chromium_patch_link, |
'skia_patch_link': self._skia_patch_link, |
+ 'benchmark_patch_link': self._benchmark_patch_link, |
'raw_csv_nopatch': self._raw_csv_nopatch, |
'raw_csv_withpatch': self._raw_csv_withpatch, |
'threshold': self._variance_threshold, |
@@ -360,6 +362,9 @@ if '__main__' == __name__: |
'', '--skia_patch_link', |
help='Link to the Skia patch used for this run.') |
option_parser.add_option( |
+ '', '--benchmark_patch_link', |
+ help='Link to the Telemetry patch used for this run.') |
+ option_parser.add_option( |
'', '--variance_threshold', |
help='The allowable variance in percentage between total values for each ' |
'field for the two CSVs.') |
@@ -427,7 +432,7 @@ if '__main__' == __name__: |
options, unused_args = option_parser.parse_args() |
if not (options.csv_file1 and options.csv_file2 and options.output_html_dir |
and options.variance_threshold and options.requester_email |
- and options.chromium_patch_link |
+ and options.chromium_patch_link and options.benchmark_patch_link |
and options.skia_patch_link and options.raw_csv_nopatch |
and options.raw_csv_withpatch and options.num_repeated |
and options.target_platform and options.pageset_type |
@@ -435,7 +440,7 @@ if '__main__' == __name__: |
and options.description): |
option_parser.error('Must specify csv_file1, csv_file2, output_html_dir, ' |
'variance_threshold, requester_email, ' |
- 'chromium_patch_link, ' |
+ 'chromium_patch_link, benchmark_patch_link, ' |
'skia_patch_link, raw_csv_nopatch, description, ' |
'raw_csv_withpatch, num_repeated, pageset_type, ' |
'chromium_hash, skia_hash and target_platform') |
@@ -443,7 +448,7 @@ if '__main__' == __name__: |
sys.exit(CsvComparer( |
options.csv_file1, options.csv_file2, options.output_html_dir, |
options.requester_email, options.chromium_patch_link, |
- options.skia_patch_link, |
+ options.skia_patch_link, options.benchmark_patch_link, |
options.variance_threshold, options.absolute_url, |
options.min_pages_in_each_field, options.discard_outliers, |
options.raw_csv_nopatch, options.raw_csv_withpatch, |