Merge pull request #123 from ricardogsilva/108-show-all-values-on-the…

…-time-series-for-raw-data Do not drop nan values for coverage times series
geobeyond · Jun 6, 2024 · d25665a · d25665a
2 parents 19c2d52 + 1c05d06
commit d25665a
Show file tree

Hide file tree

Showing 5 changed files with 159 additions and 88 deletions.
diff --git a/arpav_ppcv/operations.py b/arpav_ppcv/operations.py
@@ -400,7 +400,6 @@ def _process_coverage_data(
                 )
                 df[column_name] = loess_smoothed
         df = df.drop(columns=[base_column_name])
-        df = df.dropna()
         return df
 
 

diff --git a/arpav_ppcv/webapp/api_v2/routers/coverages.py b/arpav_ppcv/webapp/api_v2/routers/coverages.py
@@ -1,4 +1,5 @@
 import logging
+import math
 import urllib.parse
 import uuid
 from typing import (
@@ -422,7 +423,8 @@ def _serialize_dataframe(
         else:
             measurements = []
             for timestamp, value in series_measurements.items():
-                measurements.append(TimeSeriesItem(value=value, datetime=timestamp))
+                if not math.isnan(value):
+                    measurements.append(TimeSeriesItem(value=value, datetime=timestamp))
             series.append(
                 TimeSeries(
                     name=series_name,

diff --git a/tests/notebooks/timeseries-related-demo.ipynb b/tests/notebooks/timeseries-related-demo.ipynb
diff --git a/tests/notebooks/timeseries-smoothing-demo.ipynb b/tests/notebooks/timeseries-smoothing-demo.ipynb
diff --git a/tests/notebooks/timeseries-uncertainty-demo.ipynb b/tests/notebooks/timeseries-uncertainty-demo.ipynb