Skip to content
This repository was archived by the owner on Sep 17, 2025. It is now read-only.

Add helper class for stats/metrics conversion#355

Merged
c24t merged 4 commits intocensus-instrumentation:masterfrom
c24t:add-metric-descriptor-util
Oct 22, 2018
Merged

Add helper class for stats/metrics conversion#355
c24t merged 4 commits intocensus-instrumentation:masterfrom
c24t:add-metric-descriptor-util

Conversation

@c24t
Copy link
Copy Markdown
Member

@c24t c24t commented Oct 19, 2018

This diff adds a utility module for converting stats to metrics, and is the first step to producing metrics by way of views, as in https://github.com/census-instrumentation/opencensus-java/pull/1338/files.

Addresses #335.

@c24t c24t self-assigned this Oct 19, 2018
Comment thread opencensus/stats/metric_utils.py Outdated
raise AssertionError # pragma: NO COVER


def get_metric_descriptor(view):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we change it to view_to_ metric_descriptor ?

Comment thread opencensus/stats/metric_utils.py
@c24t c24t merged commit d9384fd into census-instrumentation:master Oct 22, 2018
@c24t c24t deleted the add-metric-descriptor-util branch October 22, 2018 22:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants