Merge pull request #397 from WillCodeForCats/379-problem-with-battery-reading-after-system-reboot

Use battery rating value for detection

Author: WillCodeForCats

custom_components/solaredge_modbus_multi/hub.py

@@ -1597,7 +1597,7 @@ async def init_device(self) -> None:
                         parse_modbus_string(decoder.decode_string(32)),
                     ),
                     ("B_Device_Address", decoder.decode_16bit_uint()),
-                    ("Reserved", decoder.decode_16bit_uint()),
+                    ("ignore", decoder.skip_bytes(2)),
                     ("B_RatedEnergy", decoder.decode_32bit_float()),
                     ("B_MaxChargePower", decoder.decode_32bit_float()),
                     ("B_MaxDischargePower", decoder.decode_32bit_float()),
@@ -1606,6 +1606,11 @@ async def init_device(self) -> None:
                 ]
             )
 
+            try:
+                del self.decoded_common["ignore"]
+            except KeyError:
+                pass
+
             for name, value in iter(self.decoded_common.items()):
                 if isinstance(value, float):
                     display_value = float_to_hex(value)
@@ -1644,11 +1649,11 @@ async def init_device(self) -> None:
         ].translate(ascii_ctrl_chars)
 
         if (
-            len(self.decoded_common["B_Manufacturer"]) == 0
-            or len(self.decoded_common["B_Model"]) == 0
-            or len(self.decoded_common["B_SerialNumber"]) == 0
+            float_to_hex(self.decoded_common["B_RatedEnergy"])
+            == hex(SunSpecNotImpl.FLOAT32)
+            or self.decoded_common["B_RatedEnergy"] <= 0
         ):
-            raise DeviceInvalid(f"Battery {self.battery_id} not usable.")
+            raise DeviceInvalid(f"Battery {self.battery_id} not usable (rating <=0)")
 
         self.manufacturer = self.decoded_common["B_Manufacturer"]
         self.model = self.decoded_common["B_Model"]