[Commotion-admin] [mlab-observatory] Adding sample checking module, Telescope data parser (#29)

[Collin Anderson]

> +  def add_to_counts(self, metadata, results):
> +    """Add result data to overall sample counts.
> +
> +    Args:
> +      metadata: (dict) A dictionary of metadata describing the results data.
> +        Currently the only supported value for the 'metric' key is
> +        'download_throughput'.
> +
> +      results: (list) A list of (datetime, value) pairs representing Telescope
> +        results for the given metadata.
> +
> +    Raises:
> +      ValueError: If the metadata parameter does not specify
> +        'download_throughput' as the value of the 'metric' key.
> +    """
> +    if metadata['metric_name'] != 'download_throughput':

Do you want to hard core the Observatory limitations here? Would the sample count differ for other metrics?

---
Reply to this email directly or view it on GitHub:
https://github.com/opentechinstitute/mlab-observatory/pull/29/files#r21558146
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chambana.net/pipermail/commotion-admin/attachments/20141209/2ecab3d9/attachment.html>