Index: tools/telemetry/telemetry/value/list_of_scalar_values.py |
diff --git a/tools/telemetry/telemetry/value/list_of_scalar_values.py b/tools/telemetry/telemetry/value/list_of_scalar_values.py |
index 698957ad571e6a73aaf48d0b20572788af9af4f6..65a7fc394ce97aaea6758a7dfb64009a75b39ced 100644 |
--- a/tools/telemetry/telemetry/value/list_of_scalar_values.py |
+++ b/tools/telemetry/telemetry/value/list_of_scalar_values.py |
@@ -15,9 +15,11 @@ def _Mean(values): |
class ListOfScalarValues(value_module.Value): |
def __init__(self, page, name, units, values, |
important=True, description=None, none_value_reason=None, |
- same_page_merge_policy=value_module.CONCATENATE): |
+ same_page_merge_policy=value_module.CONCATENATE, |
+ higher_is_better=None): |
super(ListOfScalarValues, self).__init__(page, name, units, important, |
- description) |
+ description, |
+ higher_is_better=higher_is_better) |
if values is not None: |
assert isinstance(values, list) |
assert len(values) > 0 |
@@ -37,14 +39,16 @@ class ListOfScalarValues(value_module.Value): |
else: |
merge_policy = 'PICK_FIRST' |
return ('ListOfScalarValues(%s, %s, %s, %s, ' |
- 'important=%s, description=%s, same_page_merge_policy=%s)') % ( |
+ 'important=%s, description=%s, same_page_merge_policy=%s, ' |
+ 'higher_is_better=%s)') % ( |
page_name, |
self.name, |
self.units, |
repr(self.values), |
self.important, |
self.description, |
- merge_policy) |
+ merge_policy, |
+ self.higher_is_better) |
def GetBuildbotDataType(self, output_context): |
if self._IsImportantGivenOutputIntent(output_context): |
@@ -98,7 +102,8 @@ class ListOfScalarValues(value_module.Value): |
values[0].values, |
important=v0.important, |
same_page_merge_policy=v0.same_page_merge_policy, |
- none_value_reason=v0.none_value_reason) |
+ none_value_reason=v0.none_value_reason, |
+ higher_is_better=v0.higher_is_better) |
assert v0.same_page_merge_policy == value_module.CONCATENATE |
return cls._MergeLikeValues(values, v0.page, v0.name) |
@@ -127,4 +132,5 @@ class ListOfScalarValues(value_module.Value): |
merged_values, |
important=v0.important, |
same_page_merge_policy=v0.same_page_merge_policy, |
- none_value_reason=none_value_reason) |
+ none_value_reason=none_value_reason, |
+ higher_is_better=v0.higher_is_better) |