diff options
| author | Caroline Tice <cmtice@google.com> | 2017-04-06 17:16:05 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-04-08 19:40:20 -0700 |
| commit | f6ef4395fe1896ba68c80e52cb24763b0fcfe7f8 (patch) | |
| tree | 7612123711db98f0746e56b66368d16b388192bd /crosperf/results_organizer.py | |
| parent | 8c18be1425c8a4ecfc059a7c637fc93f33edab1f (diff) | |
| download | toolchain-utils-f6ef4395fe1896ba68c80e52cb24763b0fcfe7f8.tar.gz | |
[toolchain-utils] Fix remaining lint errors in toolchain-utils.
In addition to fixing the lint errors, this also fixes the Python
formatting issues (ran tc_pyformat on nearly all the files).
BUG=chromium:570450
TEST=Ran all crosperf & bisect tool unit tests.
Ran afe_lock_machine.py (check machine status)
Ran full crosperf test (octane, speedometer, BootPerf) on alex.
Change-Id: Ic86f9192801ac67769f3de30f1c5f0d203ce0831
Reviewed-on: https://chromium-review.googlesource.com/471886
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Diffstat (limited to 'crosperf/results_organizer.py')
| -rw-r--r-- | crosperf/results_organizer.py | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/crosperf/results_organizer.py b/crosperf/results_organizer.py index 097c744d..62972027 100644 --- a/crosperf/results_organizer.py +++ b/crosperf/results_organizer.py @@ -69,7 +69,7 @@ def _GetNonDupLabel(max_dup, runs): new_run[key] = value else: new_key, index_str = match.groups() - added_runs[int(index_str)-1][new_key] = str(value) + added_runs[int(index_str) - 1][new_key] = str(value) new_runs.append(new_run) new_runs += added_runs return new_runs @@ -135,6 +135,7 @@ def _MakeOrganizeResultOutline(benchmark_runs, labels): result[name] = _Repeat(make_dicts, len(labels)) return result + def OrganizeResults(benchmark_runs, labels, benchmarks=None, json_report=False): """Create a dict from benchmark_runs. @@ -181,9 +182,10 @@ def OrganizeResults(benchmark_runs, labels, benchmarks=None, json_report=False): if len(cur_dict) == 1 and cur_dict['retval'] == 0: cur_dict['retval'] = 1 # TODO: This output should be sent via logger. - print("WARNING: Test '%s' appears to have succeeded but returned" - ' no results.' % benchmark.name, - file=sys.stderr) + print( + "WARNING: Test '%s' appears to have succeeded but returned" + ' no results.' % benchmark.name, + file=sys.stderr) if json_report and benchmark_run.machine: cur_dict['machine'] = benchmark_run.machine.name cur_dict['machine_checksum'] = benchmark_run.machine.checksum |