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

Unified Diff: telemetry/telemetry/value/histogram.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/failure.py ('k') | telemetry/telemetry/value/histogram_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: telemetry/telemetry/value/histogram.py
diff --git a/telemetry/telemetry/value/histogram.py b/telemetry/telemetry/value/histogram.py
index caba6911d48c8b36f9965a3150d42d7431f1be9b..15beadc4dc2bf6c74722c8cf244db58ba172a507 100644
--- a/telemetry/telemetry/value/histogram.py
+++ b/telemetry/telemetry/value/histogram.py
@@ -31,10 +31,11 @@ class HistogramValueBucket(object):
class HistogramValue(summarizable.SummarizableValue):
def __init__(self, page, name, units,
raw_value=None, raw_value_json=None, important=True,
- description=None, tir_label=None, improvement_direction=None):
+ description=None, tir_label=None, improvement_direction=None,
+ grouping_keys=None):
super(HistogramValue, self).__init__(page, name, units, important,
description, tir_label,
- improvement_direction)
+ improvement_direction, grouping_keys)
if raw_value_json:
assert raw_value == None, \
'Don\'t specify both raw_value and raw_value_json'
@@ -56,14 +57,15 @@ class HistogramValue(summarizable.SummarizableValue):
page_name = 'None'
return ('HistogramValue(%s, %s, %s, raw_json_string=%s, '
'important=%s, description=%s, tir_label=%s, '
- 'improvement_direction=%s)') % (
+ 'improvement_direction=%s, grouping_keys=%s)') % (
page_name,
self.name, self.units,
self.ToJSONString(),
self.important,
self.description,
self.tir_label,
- self.improvement_direction)
+ self.improvement_direction,
+ self.grouping_keys)
def GetBuildbotDataType(self, output_context):
if self._IsImportantGivenOutputIntent(output_context):
@@ -111,8 +113,6 @@ class HistogramValue(summarizable.SummarizableValue):
if 'improvement_direction' in value_dict:
kwargs['improvement_direction'] = value_dict['improvement_direction']
- if 'tir_label' in value_dict:
- kwargs['tir_label'] = value_dict['tir_label']
return HistogramValue(**kwargs)
@@ -125,7 +125,8 @@ class HistogramValue(summarizable.SummarizableValue):
raw_value_json=histogram_util.AddHistograms(
[v.ToJSONString() for v in values]),
important=v0.important, tir_label=v0.tir_label,
- improvement_direction=v0.improvement_direction)
+ improvement_direction=v0.improvement_direction,
+ grouping_keys=v0.grouping_keys)
@classmethod
def MergeLikeValuesFromDifferentPages(cls, values):
« no previous file with comments | « telemetry/telemetry/value/failure.py ('k') | telemetry/telemetry/value/histogram_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698