From 659d7015dbf4c21d1312213108b5feff036d739a Mon Sep 17 00:00:00 2001 From: Andrey Logvin Date: Tue, 24 Nov 2020 15:12:25 +0000 Subject: [PATCH] Replace all infinity values before uploading to the dashoard No-Presubmit: True Bug: webrtc:12224 Change-Id: I4a7a33bedb5449d7e36293e0c6f47e9cd4d0235a Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/195200 Reviewed-by: Mirko Bonadei Commit-Queue: Andrey Logvin Cr-Commit-Position: refs/heads/master@{#32685} --- tools_webrtc/perf/catapult_uploader.py | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/tools_webrtc/perf/catapult_uploader.py b/tools_webrtc/perf/catapult_uploader.py index 12713c56f5..b064071755 100644 --- a/tools_webrtc/perf/catapult_uploader.py +++ b/tools_webrtc/perf/catapult_uploader.py @@ -172,17 +172,18 @@ def _CheckFullUploadInfo(url, upload_token, # TODO(https://crbug.com/1029452): HACKHACK # Remove once we have doubles in the proto and handle -infinity correctly. def _ApplyHacks(dicts): + def _NoInf(value): + if value == float('inf'): + return histogram.JS_MAX_VALUE + if value == float('-inf'): + return -histogram.JS_MAX_VALUE + return value + for d in dicts: if 'running' in d: - - def _NoInf(value): - if value == float('inf'): - return histogram.JS_MAX_VALUE - if value == float('-inf'): - return -histogram.JS_MAX_VALUE - return value - d['running'] = [_NoInf(value) for value in d['running']] + if 'sampleValues' in d: + d['sampleValues'] = [_NoInf(value) for value in d['sampleValues']] return dicts