prometheus-api-client-python icon indicating copy to clipboard operation
prometheus-api-client-python copied to clipboard

Handling of NaN values in metric aggregations

Open GPalaciosFdez opened this issue 2 years ago • 1 comments

Describe the bug Not really sure if a bug or a design decision but I could not find this case covered in the tests.

When using numpy to calculate the different operations in get_metric_aggregation the functions chosen will return NaN if there is a value in the metric that is NaN.

For example for:

To Reproduce Steps to reproduce the behavior:

  1. Pass a metric with a NaN value and several numerical values to get_metric_aggregation
  2. Check the result and you should see that it is NaN

Expected behavior Handle NaN values to return a statistically meaningful number

Additional context I could submit a PR if you think this is something that can be changed. It could even be possible to pass a boolean flag handle_nans as an argument to get_metric_aggregation to choose the behavior.

GPalaciosFdez avatar Jun 08 '23 11:06 GPalaciosFdez

Hi :)

Any comments about this issue?

GPalaciosFdez avatar Jul 13 '23 07:07 GPalaciosFdez