Fix Metric repr for ValueDistributions (#559)

Author: c24t

opencensus/metrics/export/metric.py

@@ -43,24 +43,10 @@ def __init__(self, descriptor, time_series):
         self._check_type()
 
     def __repr__(self):
-        # Get a condensed description of a list of time series like:
-        # (k1="v11", k2="v21", [12]), (k1="v21", k2="v22", [34])
-        lks = [lk.key for lk in self.descriptor.label_keys]
-        labeled_pvs = []
-        for ts in self.time_series:
-            labels = ", ".join('{}="{}"'.format(lk, lv)
-                               for lk, lv in zip(lks, ts.label_values))
-            pvs = [point.value.value if point.value else None
-                   for point in ts.points]
-            labeled_pvs.append((labels, pvs))
-        short_ts_repr = ", ".join("({}, {})".format(ll, pvs)
-                                  for ll, pvs in labeled_pvs)
-
-        # E.g. 'Metric((k1="v11", k2="v21", [12]), descriptor.name="name")'
-        return ('{}({}, descriptor.name="{}")'
+        return ('{}(time_series={}, descriptor.name="{}")'
                 .format(
                     type(self).__name__,
-                    short_ts_repr,
+                    "<{} TimeSeries>".format(len(self.time_series)),
                     self.descriptor.name,
                 ))
 

opencensus/metrics/export/time_series.py

@@ -47,11 +47,15 @@ def __init__(self, label_values, points, start_timestamp):
         self._start_timestamp = start_timestamp
 
     def __repr__(self):
+        points_repr = '[{}]'.format(
+            ', '.join(repr(point.value) for point in self.points))
+
+        lv_repr = tuple(lv.value for lv in self.label_values)
         return ('{}({}, label_values={}, start_timestamp={})'
                 .format(
                     type(self).__name__,
-                    [point.value.value for point in self.points],
-                    self.label_values,
+                    points_repr,
+                    lv_repr,
                     self.start_timestamp
                 ))
 

opencensus/metrics/export/value.py

@@ -110,6 +110,13 @@ def __init__(self, value, timestamp, attachments):
         self._timestamp = timestamp
         self._attachments = attachments
 
+    def __repr__(self):
+        return ("{}({})"
+                .format(
+                    type(self).__name__,
+                    self.value,
+                ))
+
     @property
     def value(self):
         return self._value
@@ -138,6 +145,13 @@ def __init__(self, count, exemplar=None):
         self._count = count
         self._exemplar = exemplar
 
+    def __repr__(self):
+        return ("{}({})"
+                .format(
+                    type(self).__name__,
+                    self.count,
+                ))
+
     @property
     def count(self):
         return self._count
@@ -186,6 +200,13 @@ class BucketOptions(object):
     def __init__(self, type_=None):
         self._type = type_
 
+    def __repr__(self):
+        return ("{}({})"
+                .format(
+                    type(self).__name__,
+                    self.type_,
+                ))
+
     @property
     def type_(self):
         return self._type
@@ -253,6 +274,18 @@ def __init__(self,
         self._bucket_options = bucket_options
         self._buckets = buckets
 
+    def __repr__(self):
+        try:
+            bounds = self.bucket_options.type_.bounds,
+        except AttributeError:
+            bounds = None
+
+        return ("{}({})"
+                .format(
+                    type(self).__name__,
+                    bounds
+                ))
+
     @property
     def count(self):
         return self._count