Handle missing battery capacity in SEMS API response gracefully

Enclosed the retrieval in a try-except block to catch KeyError and TypeError. Added a warning log when defaulting to a predefined value of 10 kWh

Author: felixschndr

source/inverter.py

@@ -18,6 +18,8 @@ def __init__(self):
 
         self.sems_portal_api_handler = SemsPortalApiHandler()
         self.battery_capacity = None
+        # We have to pull the battery capacity at startup since there are functions here that require it which are
+        # called when using the bash script to control the inverter manually
         self.update_battery_capacity()
 
     @property

source/sems_portal_api_handler.py

@@ -376,4 +376,12 @@ def get_battery_capacity(self) -> EnergyAmount:
 
         self.log.trace(f"Retrieved data: {response}")
 
-        return EnergyAmount.from_kilo_watt_hours(response["data"]["info"]["battery_capacity"])
+        try:
+            return EnergyAmount.from_kilo_watt_hours(response["data"]["info"]["battery_capacity"])
+        except (KeyError, TypeError):
+            # This is not that bad as we pull the battery capacity every iteration
+            # If this is the first time after starting we don't even use the value
+            self.log.warning(
+                "Unable to retrieve battery capacity from SEMSPORTAL API, using a default value", exc_info=True
+            )
+            return EnergyAmount.from_kilo_watt_hours(10)