Index: bench/check_bench_regressions.py |
diff --git a/bench/check_bench_regressions.py b/bench/check_bench_regressions.py |
index 48ce180a39687b22ef78f10094790e5734554bfa..8f648dc2b0f23cdbb05b721619c10163af8e96d4 100644 |
--- a/bench/check_bench_regressions.py |
+++ b/bench/check_bench_regressions.py |
@@ -80,35 +80,6 @@ class Label: |
hash(self.time_type) ^ |
hash(frozenset(self.settings.iteritems()))) |
-def parse_dir(directory, default_settings, revision, rep): |
- """Parses bench data from bench logs files. |
- revision can be either svn revision or git commit hash. |
- """ |
- revision_data_points = [] # list of BenchDataPoint |
- file_list = os.listdir(directory) |
- file_list.sort() |
- for bench_file in file_list: |
- scalar_type = None |
- # Scalar type, if any, is in the bench filename after revision |
- if (len(revision) > MAX_SVN_REV_LENGTH and |
- bench_file.startswith('bench_' + revision + '_')): |
- # The revision is GIT commit hash. |
- scalar_type = bench_file[len(revision) + len('bench_') + 1:] |
- elif (bench_file.startswith('bench_r' + revision + '_') and |
- revision.isdigit()): |
- # The revision is SVN number |
- scalar_type = bench_file[len(revision) + len('bench_r') + 1:] |
- else: |
- continue |
- |
- file_handle = open(directory + '/' + bench_file, 'r') |
- |
- default_settings['scalar'] = scalar_type |
- revision_data_points.extend( |
- bench_util.parse(default_settings, file_handle, rep)) |
- file_handle.close() |
- return revision_data_points |
- |
def create_bench_dict(revision_data_points): |
"""Convert current revision data into a dictionary of line data. |
@@ -206,6 +177,7 @@ def check_expectations(lines, expectations, key_suffix): |
if outputs: |
raise Exception('\n'.join(outputs)) |
+ |
def main(): |
"""Parses command line and checks bench expectations.""" |
try: |
@@ -248,10 +220,7 @@ def main(): |
platform_and_alg = bot + '-' + rep |
- data_points = parse_dir(directory, |
- {}, # Sets default settings to empty. |
- rev, |
- rep) |
+ data_points = bench_util.parse_skp_bench_data(directory, rev, rep) |
bench_dict = create_bench_dict(data_points) |