Only use raw value in update_accum

Author: WillCodeForCats

custom_components/solaredge_modbus_multi/helpers.py

@@ -10,18 +10,18 @@ def watts_to_kilowatts(value):
 def parse_modbus_string(s: str) -> str:
     return s.decode(encoding="utf-8", errors="ignore").replace("\x00", "").rstrip()
 
-def update_accum(self, raw: int, current: int) -> None:
+def update_accum(self, accum_value: int) -> None:
 
     if self.last is None:
         self.last = 0
 
-    if not raw > 0:
+    if not accum_value > 0:
         raise ValueError(f"update_accum must be non-zero value.")
 
-    if current >= self.last:
+    if accum_value >= self.last:
         # doesn't account for accumulator rollover, but it would probably take
         # several decades to roll over to 0 so we'll worry about it later
-        self.last = current
-        return current    
+        self.last = accum_value
+        return accum_value    
     else:
         raise ValueError(f"update_accum must be an increasing value.")

custom_components/solaredge_modbus_multi/sensor.py

@@ -839,10 +839,9 @@ def native_value(self):
 
             else:
                 value = scale_factor(self._platform.decoded_model[model_key], self._platform.decoded_model['AC_Energy_WH_SF'])
-                value_kw = watts_to_kilowatts(value)
 
                 try:
-                    return update_accum(self, value, value_kw)
+                    return watts_to_kilowatts(update_accum(self, value))
                 except:
                     return None