Merge pull request #630 from WillCodeForCats/handle-inverter-no-response

Continue on inverter no response when reading grid on/off

Author: WillCodeForCats

custom_components/solaredge_modbus_multi/binary_sensor.py

@@ -33,8 +33,7 @@ async def async_setup_entry(
         if hub.option_detect_extras and inverter.advanced_power_control:
             entities.append(AdvPowerControlEnabled(inverter, config_entry, coordinator))
 
-        if hub.option_detect_extras:
-            entities.append(GridStatusOnOff(inverter, config_entry, coordinator))
+        entities.append(GridStatusOnOff(inverter, config_entry, coordinator))
 
     if entities:
         async_add_entities(entities)

custom_components/solaredge_modbus_multi/hub.py

@@ -460,8 +460,8 @@ async def connect(self) -> None:
         """Connect to inverter."""
 
         if self._client is None:
-            _LOGGER.debug(f"New client object for {self._host}:{self._port}")
             _LOGGER.debug(
+                "New AsyncModbusTcpClient: "
                 f"reconnect_delay={self._mb_reconnect_delay} "
                 f"reconnect_delay_max={self._mb_reconnect_delay_max} "
                 f"retry_on_empty={self._mb_retry_on_empty} "
@@ -476,6 +476,7 @@ async def connect(self) -> None:
                 timeout=self._mb_timeout,
             )
 
+        _LOGGER.debug((f"Connecting to {self._host}:{self._port} ..."))
         await self._client.connect()
 
     def disconnect(self, clear_client: bool = False) -> None:
@@ -1323,7 +1324,14 @@ async def read_modbus_data(self) -> None:
                 )
                 self._grid_status = True
 
-            except ModbusIllegalAddress:
+            except (ModbusIllegalAddress, ModbusIOException) as e:
+
+                if (
+                    type(e) is ModbusIOException
+                    and "No response recieved after" not in e
+                ):
+                    raise
+
                 try:
                     del self.decoded_model["I_Grid_Status"]
                 except KeyError:
@@ -1332,9 +1340,12 @@ async def read_modbus_data(self) -> None:
                 self._grid_status = False
 
                 _LOGGER.debug(
-                    (f"I{self.inverter_unit_id}: " "Grid On/Off NOT available")
+                    (f"I{self.inverter_unit_id}: Grid On/Off NOT available: {e}")
                 )
 
+                if not self.hub.is_connected:
+                    await self.hub.connect()
+
             except ModbusIOError:
                 raise ModbusReadError(
                     f"No response from inverter ID {self.inverter_unit_id}"

custom_components/solaredge_modbus_multi/manifest.json

@@ -10,5 +10,5 @@
   "issue_tracker": "https://github.com/WillCodeForCats/solaredge-modbus-multi/issues",
   "loggers": ["custom_components.solaredge_modbus_multi"],
   "requirements": ["pymodbus>=3.6.6"],
-  "version": "2.4.18"
+  "version": "2.4.19-pre.2"
 }