Guard against repeated collection from the same timestamp

Author: bouwew

plugwise_usb/nodes/helpers/pulses.py

@@ -93,6 +93,7 @@ def __init__(self, mac: str) -> None:
         self._prev_pulses_consumption: int = 0
         self._pulses_consumption: int | None = None
         self._prev_pulses_production: int = 0
+        self._prev_timestamp: datetime | None = None
         self._pulses_production: int | None = None
         self._pulses_timestamp: datetime | None = None
 
@@ -188,9 +189,13 @@ def collected_pulses(
         timestamp: datetime | None = None
         if is_consumption and self._pulses_consumption is not None:
             timestamp = self._pulses_timestamp
+            if self._prev_timestamp == timestamp:
+                return
+
             delta_cons_pulses = self._pulses_consumption - self._prev_pulses_consumption
             pulses = self._prev_pulses_consumption + delta_cons_pulses
             self._prev_pulses_consumption = pulses
+            self._prev_timestamp = timestamp
             if reset:
                 pulses = delta_cons_pulses
                 self._prev_pulses_consumption = 0
@@ -203,6 +208,7 @@ def collected_pulses(
             delta_prod_pulses = self._pulses_production - self._prev_pulses_production
             pulses = self._prev_pulses_production + delta_prod_pulses
             self._prev_pulses_production = pulses
+            self._prev_timestamp = timestamp
             if reset:
                 pulses = delta_prod_pulses
                 self._prev_pulses_production = 0