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

Unified Diff: telemetry/telemetry/value/summarizable.py

Issue 1685683003: Implement Timeline Based Measurement v2 (Closed) Base URL: git@github.com:catapult-project/catapult.git@new_style_results
Patch Set: fix vinn tests Created 4 years, 10 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
« no previous file with comments | « telemetry/telemetry/value/skip.py ('k') | telemetry/telemetry/value/summarizable_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: telemetry/telemetry/value/summarizable.py
diff --git a/telemetry/telemetry/value/summarizable.py b/telemetry/telemetry/value/summarizable.py
index 828099b0beb57365d9f39ddd7469b6554fc90829..9cfcf46f9c2419224946eb0f2b1085aa4f948b45 100644
--- a/telemetry/telemetry/value/summarizable.py
+++ b/telemetry/telemetry/value/summarizable.py
@@ -9,10 +9,10 @@ from telemetry.value import (improvement_direction
class SummarizableValue(value_module.Value):
def __init__(self, page, name, units, important, description, tir_label,
- improvement_direction):
+ improvement_direction, grouping_keys):
"""A summarizable value result from a test."""
super(SummarizableValue, self).__init__(
- page, name, units, important, description, tir_label)
+ page, name, units, important, description, tir_label, grouping_keys)
# TODO(eakuefner): uncomment this assert after Telemetry clients are fixed.
# Note: Telemetry unittests satisfy this assert.
# assert improvement_direction_module.IsValid(improvement_direction)
« no previous file with comments | « telemetry/telemetry/value/skip.py ('k') | telemetry/telemetry/value/summarizable_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698