frequenz-floss / frequenz-client-reporting-python

A python client for the Frequenz Reporting API
MIT License
0 stars 5 forks source link

Support for full metric sample variant #21

Open cwasicki opened 4 months ago

cwasicki commented 4 months ago

What's needed?

Currently the client assumes that all metric samples are of SimpleMetricSample type, but the MetricSampleVariant can also be of AggregatedMetricSample. The client should support this type too.

Proposed solution

The AggregatedMetricSample contains a float, two optional floats and a list of floats. The first iteration could just use the single float value and use it in the same way as currently simple metric samples are used.

Use cases

No response

Alternatives and workarounds

No response

Additional context

No response