Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(207)

Unified Diff: tools/telemetry/telemetry/value/summary.py

Issue 636203002: Task Execution Duration metrics added as a new benchmark (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Small code layout refactor to aid readability Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/telemetry/telemetry/value/summary.py
diff --git a/tools/telemetry/telemetry/value/summary.py b/tools/telemetry/telemetry/value/summary.py
index 13e8ac20dfc52ba5b387fa9940d4d82099cbc6f5..d3a52930265931d4b1c6b24c424d1a9888562618 100644
--- a/tools/telemetry/telemetry/value/summary.py
+++ b/tools/telemetry/telemetry/value/summary.py
@@ -42,6 +42,8 @@ class Summary(object):
self._computed_per_page_values = []
self._computed_summary_values = []
self._interleaved_computed_per_page_values_and_summaries = []
+ self._only_summary = any(isinstance(v, skip.SkipValue) and v.only_summaries
+ for v in all_page_specific_values)
self._ComputePerPageValues(all_page_specific_values)
@property
@@ -132,7 +134,7 @@ class Summary(object):
num_successful_pages_for_value_name[value_name])
for per_page_value in sorted_per_page_values:
self._ComputePerPageValue(per_page_value,
- num_successful_pages_for_this_value_name)
+ num_successful_pages_for_this_value_name)
# Output the combined values.
merged_pages_value = merged_pages_value_by_value_name.get(value_name,
@@ -147,7 +149,10 @@ class Summary(object):
# If there were any page errors, we typically will print nothing.
#
# Note: this branch is structured less-densely to improve legibility.
- if num_successful_pages_for_this_value_name > 1:
+
+ if self._only_summary:
+ should_print = False
+ elif num_successful_pages_for_this_value_name > 1:
should_print = True
elif (self.had_failures and
num_successful_pages_for_this_value_name == 1):
« tools/perf/measurements/task_execution_time.py ('K') | « tools/telemetry/telemetry/value/skip.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698