Treat NaNs as missing values for gaps in ring buffer

Signed-off-by: cwasicki <[email protected]>

Author: cwasicki

RELEASE_NOTES.md

@@ -20,6 +20,7 @@
 
 - A new class `Fuse` has been added to represent fuses. This class has a member variable `max_current` which represents the maximum current that can course through the fuse. If the current flowing through a fuse is greater than this limit, then the fuse will break the circuit.
 
+- NaN values are treated as missing when gaps are determined in the `OrderedRingBuffer`.
 
 ## Bug Fixes
 

src/frequenz/sdk/timeseries/_ringbuffer/buffer.py

@@ -108,6 +108,17 @@ def gaps(self) -> List[Gap]:
         """
         return self._gaps
 
+    def has_value(self, sample: Sample[QuantityT]) -> bool:
+        """Check if a sample has a value and it's not NaN.
+
+        Args:
+            sample: sample to check.
+
+        Returns:
+            True if the sample has a value and it's not NaN.
+        """
+        return not (sample.value is None or sample.value.isnan())
+
     @property
     def maxlen(self) -> int:
         """Get the max length.
@@ -154,10 +165,14 @@ def update(self, sample: Sample[QuantityT]) -> None:
         )
 
         # Update data
-        value: float = np.nan if sample.value is None else sample.value.base_value
+        if self.has_value(sample):
+            assert sample.value is not None
+            value = sample.value.base_value
+        else:
+            value = np.nan
         self._buffer[self.datetime_to_index(timestamp)] = value
 
-        self._update_gaps(timestamp, prev_newest, sample.value is None)
+        self._update_gaps(timestamp, prev_newest, not self.has_value(sample))
 
     @property
     def time_bound_oldest(self) -> datetime:

tests/timeseries/test_ringbuffer.py

@@ -229,16 +229,16 @@ def test_gaps() -> None:
     assert len(buffer.gaps) == 2
 
     buffer.update(Sample(dt(3), Quantity(np.nan)))
-    assert len(buffer) == 4  # should be 3
-    assert len(buffer.gaps) == 1  # should be 2
+    assert len(buffer) == 3
+    assert len(buffer.gaps) == 2
 
     buffer.update(Sample(dt(2), Quantity(np.nan)))
-    assert len(buffer) == 4  # should be 2
-    assert len(buffer.gaps) == 1  # should be 2
+    assert len(buffer) == 2
+    assert len(buffer.gaps) == 2
 
     buffer.update(Sample(dt(3), Quantity(3)))
-    assert len(buffer) == 4  # should be 3
-    assert len(buffer.gaps) == 1  # should be 2
+    assert len(buffer) == 3
+    assert len(buffer.gaps) == 2
 
     buffer.update(Sample(dt(2), Quantity(2)))
     assert len(buffer) == 4