diff options
author | Denis Nikitin <denik@google.com> | 2019-07-25 16:55:37 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-07-26 22:54:37 -0700 |
commit | e2fe0ae6ceedeb93847b66560c2c010078b07def (patch) | |
tree | e3a2ccedb272d030a2f5baa2221f7f9b5f22675c /crosperf/results_organizer.py | |
parent | d3466be7b6dbf2153f093b5a03436561a659a9bd (diff) | |
download | toolchain-utils-e2fe0ae6ceedeb93847b66560c2c010078b07def.tar.gz |
crosperf: Fix pylint warnings
Pylint throws a bunch of warnings on results_cache.py and
results_organizer.py (other source files not checked).
The change fixes most warnings EXCEPT docstring which would require more
elaboration.
BUG=none
TEST=presubmit passess
Change-Id: I4e48d94c9b430d8b224e70e9267abe3795bff7cc
Reviewed-on: https://chromium-review.googlesource.com/1719770
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Ready: Denis Nikitin <denik@chromium.org>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'crosperf/results_organizer.py')
-rw-r--r-- | crosperf/results_organizer.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/crosperf/results_organizer.py b/crosperf/results_organizer.py index 2739f739..fb7c60dc 100644 --- a/crosperf/results_organizer.py +++ b/crosperf/results_organizer.py @@ -2,6 +2,7 @@ # Copyright (c) 2013 The Chromium OS Authors. All rights reserved. # Use of this source code is governed by a BSD-style license that can be # found in the LICENSE file. + """Parse data from benchmark_runs for tabulator.""" from __future__ import print_function @@ -45,7 +46,7 @@ def _GetMaxDup(data): def _Repeat(func, times): """Returns the result of running func() n times.""" - return [func() for _ in xrange(times)] + return [func() for _ in range(times)] def _DictWithReturnValues(retval, pass_fail): @@ -170,6 +171,7 @@ def OrganizeResults(benchmark_runs, labels, benchmarks=None, json_report=False): label_names = [label.name for label in labels] label_indices = {name: i for i, name in enumerate(label_names)} summary_file = _ReadSummaryFile(sys.argv[0]) + if benchmarks is None: benchmarks = [] |