diff options
| author | cmtice <cmtice@google.com> | 2013-12-10 16:30:02 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-12-13 22:27:10 +0000 |
| commit | af5f70ecd0f07c76eb83eeb3e15d4bb168f9e8b1 (patch) | |
| tree | 20895ad090a3e96b7d2b4c5699683aec927c4bca /crosperf/results_cache.py | |
| parent | fb27ee972816c8c2fddf2540792131fad2a25d0d (diff) | |
| download | toolchain-utils-af5f70ecd0f07c76eb83eeb3e15d4bb168f9e8b1.tar.gz | |
Fix bug with Telemetry results names.
Some Telemetry tests return results that have the same 'description'
name, but different 'graph' names. We were only using the description
names, so we were losing some Telemetry measurements in our reports.
This CL fixes that.
BUG=None
TEST=Ran this on with smoothness.top_25. Was missing some results
before this change; after, the results are all there.
Change-Id: Ibcbe31da5f2860c494cbc8364b0a720d1c50169c
Reviewed-on: https://chrome-internal-review.googlesource.com/150215
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf/results_cache.py')
| -rw-r--r-- | crosperf/results_cache.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/crosperf/results_cache.py b/crosperf/results_cache.py index 3ad2df75..64c22138 100644 --- a/crosperf/results_cache.py +++ b/crosperf/results_cache.py @@ -81,8 +81,8 @@ class Result(object): lines = data_file.readlines() for line in lines: tmp_dict = json.loads(line) - key = tmp_dict['description'] - val = tmp_dict['value'] + key = tmp_dict["graph"] + "__" + tmp_dict["description"] + val = tmp_dict["value"] keyvals_dict[key] = val return keyvals_dict |