Merge pull request #828 from WillCodeForCats/code-quality

Explicitly set pymodbus retries value

Author: WillCodeForCats

custom_components/solaredge_modbus_multi/const.py

@@ -70,6 +70,7 @@ class ModbusDefaults(IntEnum):
     """
 
     Timeout = 3  # Timeout for a request, in seconds.
+    Retries = 3  # Max number of retries per request.
     ReconnectDelay = 0  # Minimum in seconds.milliseconds before reconnecting.
     ReconnectDelayMax = 3.0  # Maximum in seconds.milliseconds before reconnecting.
 

custom_components/solaredge_modbus_multi/hub.py

@@ -165,6 +165,9 @@ def __init__(
         self._mb_timeout = self._yaml_config.get("modbus", {}).get(
             "timeout", ModbusDefaults.Timeout
         )
+        self._mb_retries = self._yaml_config.get("modbus", {}).get(
+            "retries", ModbusDefaults.Retries
+        )
         self._id = entry_data[CONF_NAME].lower()
         self._lock = asyncio.Lock()
         self.inverters = []
@@ -486,6 +489,7 @@ async def connect(self) -> None:
                 reconnect_delay=self._mb_reconnect_delay,
                 reconnect_delay_max=self._mb_reconnect_delay_max,
                 timeout=self._mb_timeout,
+                retries=self._mb_retries,
             )
 
         _LOGGER.debug((f"Connecting to {self._host}:{self._port} ..."))