gen_bench_expectations to generate bench expectations files.

bench/gen_bench_expectations.py

Index: bench/gen_bench_expectations.py
diff --git a/bench/gen_bench_expectations.py b/bench/gen_bench_expectations.py
new file mode 100644
index 0000000000000000000000000000000000000000..f4af3da29b11cae8b0d25a9d3eb47a0912af8dd1
--- /dev/null
+++ b/bench/gen_bench_expectations.py
@@ -0,0 +1,109 @@
+#!/usr/bin/env python
+# Copyright (c) 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+""" Generate bench_expectations file from a given set of bench data files. """
+
+import argparse
+import bench_util
+import os
+import re
+import sys
+
+# Parameters for calculating bench ranges.
+RANGE_RATIO = 1.0  # Ratio of range for upper and lower bounds.
+ABS_ERR = 1.0  # Additional allowed error in milliseconds.
+
+# List of bench configs to monitor. Ignore all other configs.
+CONFIGS_TO_INCLUDE = ['simple_viewport_1000x1000',
+                      'simple_viewport_1000x1000_gpu',
+                      'simple_viewport_1000x1000_scalar_1.100000',
+                      'simple_viewport_1000x1000_scalar_1.100000_gpu',
+                     ]
+
+
+def compute_ranges(benches):
+  """Given a list of bench numbers, calculate the alert range.
+
+  Args:
+    benches: a list of float bench values.
+
+  Returns:
+    a list of float [lower_bound, upper_bound].
+  """
+  minimum = min(benches)
+  maximum = max(benches)
+  diff = maximum - minimum
+
+  return [minimum - diff * RANGE_RATIO - ABS_ERR,
+          maximum + diff * RANGE_RATIO + ABS_ERR]
+
+
+def create_expectations_dict(revision_data_points):
+  """Convert list of bench data points into a dictionary of expectations data.
+
+  Args:
+    revision_data_points: a list of BenchDataPoint objects.
+
+  Returns:
+    a dictionary of this form:
+        keys = tuple of (config, bench) strings.
+        values = list of float [expected, lower_bound, upper_bound] for the key.
+  """
+  bench_dict = {}
+  for point in revision_data_points:
+    if (point.time_type or  # Not walltime which has time_type ''

[borenet, 2014/03/17 17:43:05]

This seems confusing to me, but I guess it's okay since we're already using this
format...

[benchen, 2014/03/17 19:02:40]

this is legacy format. I think it meant "default". will leave as is now.
On 2014/03/17 17:43:05, borenet wrote:

+        not point.config in CONFIGS_TO_INCLUDE):
+      continue
+    key = (point.config, point.bench)
+    if key in bench_dict:
+      raise Exception('Duplicate bench entry: ' + str(key))
+    bench_dict[key] = [point.time] + compute_ranges(point.per_iter_time)
+
+  return bench_dict
+
+
+def main():
+    """Reads bench data points, then calculate and export expectations.
+    """
+    parser = argparse.ArgumentParser()
+    parser.add_argument(
+        '-a', '--representation_alg', default='25th',
+        help='bench representation algorithm to use, see bench_util.py.')
+    parser.add_argument(
+        '-b', '--builder', required=True,
+        help='name of the builder whose bench ranges we are computing.')
+    parser.add_argument(
+        '-d', '--input_dir', required=True,
+        help='a directory containing bench data files.')
+    parser.add_argument(
+        '-o', '--output_file', required=True,
+        help='file path and name for storing the output bench expectations.')
+    parser.add_argument(
+        '-r', '--git_revision', required=True,
+        help='the git hash to indicate the revision of input data to use.')
+    args = parser.parse_args()
+
+    builder = args.builder
+
+    data_points = bench_util.parse_skp_bench_data(
+        args.input_dir, args.git_revision, args.representation_alg)
+
+    expectations_dict = create_expectations_dict(data_points)
+
+    out_lines = []
+    keys = expectations_dict.keys()
+    keys.sort()
+    for (config, bench) in keys:
+      (expected, lower_bound, upper_bound) = expectations_dict[(config, bench)]
+      out_lines.append('%s_%s_,%s-%s,%s,%s,%s' % (
+          bench, config, builder, args.representation_alg, expected,
+          lower_bound, upper_bound))

[borenet, 2014/03/17 17:43:05]

Can we use a format dict here for clarity?

'%(bench)s_%(config)s_,%(builder)s-%(representation)s,%(expected)s,%(lower_bound)s,%(upper_bound)s'
% {
  'bench': bench,
  ....
}

[benchen, 2014/03/17 19:02:40]

Done.

+
+    with open(args.output_file, 'w') as file_handle:
+      file_handle.write('\n'.join(out_lines))
+
+
+if __name__ == "__main__":
+    main()