+ Added all SAVE units information for calculate power consumption

* Fixed changing unit model cause to crash
+ Added model images for future use
* Cleanup & Stabilize

Author: AN3Orik

custom_components/systemair/__init__.py

@@ -47,12 +47,18 @@ async def async_setup_entry(hass: HomeAssistant, entry: SystemairConfigEntry) ->
         model=model,
     )
 
+    entry.async_on_unload(entry.add_update_listener(async_options_update_listener))
+
     await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
-    entry.async_on_unload(entry.add_update_listener(async_reload_entry))
 
     return True
 
 
+async def async_options_update_listener(_hass: HomeAssistant, entry: ConfigEntry) -> None:
+    """Handle options update."""
+    entry.runtime_data.model = entry.options[CONF_MODEL]
+
+
 async def async_unload_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
     """Handle removal of an entry."""
     unload_ok = await hass.config_entries.async_unload_platforms(entry, PLATFORMS)

custom_components/systemair/api.py

@@ -8,6 +8,7 @@
 from pymodbus.exceptions import ConnectionException
 
 from .const import LOGGER
+from .modbus import parameter_map
 
 MODBUS_DEVICE_BUSY_EXCEPTION = 6
 MODBUS_GATEWAY_TARGET_FAILED_TO_RESPOND = 11
@@ -18,7 +19,7 @@ class ModbusConnectionError(Exception):
 
 
 class SystemairVSRModbusClient:
-    """Provides a client for interacting with a Systemair VSR unit via Modbus."""
+    """Provides a client for interacting with a Systemair VSR unit."""
 
     def __init__(self, host: str, port: int, slave_id: int, timeout: int = 5) -> None:
         """Initialize the Modbus client."""
@@ -55,6 +56,20 @@ async def stop(self) -> None:
             await self._close_connection()
             LOGGER.info("Systemair Modbus client worker stopped.")
 
+    async def test_connection(self) -> bool:
+        """Start, test a single read, and stop the client to validate connection."""
+        try:
+            await self.start()
+            test_register_1based = parameter_map["REG_TC_SP"].register
+            await self._queue_request("read", address=test_register_1based - 1, count=1)
+        except (TimeoutError, ModbusConnectionError) as e:
+            LOGGER.error("Failed to connect during test: %s", e)
+            return False
+        else:
+            return True
+        finally:
+            await self.stop()
+
     async def _ensure_connected(self) -> None:
         """Ensure the client is connected, establishing connection if needed."""
         if self._client and self._client.connected:

custom_components/systemair/config_flow.py

@@ -28,10 +28,10 @@ class SystemairVSRConfigFlow(config_entries.ConfigFlow, domain=DOMAIN):
     @staticmethod
     @callback
     def async_get_options_flow(
-        config_entry: config_entries.ConfigEntry,
+        _config_entry: config_entries.ConfigEntry,
     ) -> SystemairOptionsFlowHandler:
         """Get the options flow for this handler."""
-        return SystemairOptionsFlowHandler(config_entry)
+        return SystemairOptionsFlowHandler()
 
     async def _validate_connection(self, user_input: dict) -> None:
         """Validate the connection to the VSR unit."""
@@ -55,7 +55,7 @@ async def async_step_user(self, user_input: dict | None = None) -> config_entrie
                 errors["base"] = "cannot_connect"
             except (TimeoutError, OSError) as e:
                 LOGGER.exception("Unexpected exception: %s", e)
-                errors["base"] = "cannot_connect"
+                errors["base"] = "unknown"
             else:
                 unique_id = f"{user_input[CONF_HOST]}:{user_input[CONF_PORT]}"
                 await self.async_set_unique_id(unique_id)
@@ -70,7 +70,7 @@ async def async_step_user(self, user_input: dict | None = None) -> config_entrie
                     vol.Required(CONF_HOST): selector.TextSelector(),
                     vol.Required(CONF_PORT, default=DEFAULT_PORT): vol.Coerce(int),
                     vol.Required(CONF_SLAVE_ID, default=DEFAULT_SLAVE_ID): vol.Coerce(int),
-                    vol.Required(CONF_MODEL): selector.SelectSelector(
+                    vol.Required(CONF_MODEL, default=next(iter(MODEL_SPECS))): selector.SelectSelector(
                         selector.SelectSelectorConfig(
                             options=list(MODEL_SPECS.keys()),
                             mode=selector.SelectSelectorMode.DROPDOWN,
@@ -85,6 +85,11 @@ async def async_step_user(self, user_input: dict | None = None) -> config_entrie
 class SystemairOptionsFlowHandler(config_entries.OptionsFlow):
     """Handle an options flow for Systemair."""
 
+    @property
+    def config_entry(self) -> config_entries.ConfigEntry:
+        """Return the config entry for this flow."""
+        return self.hass.config_entries.async_get_entry(self.handler)
+
     async def async_step_init(self, user_input: dict | None = None) -> config_entries.ConfigFlowResult:
         """Manage the options."""
         if user_input is not None:

custom_components/systemair/const.py

@@ -15,14 +15,53 @@
 
 # --- Power Specs for different models ---
 MODEL_SPECS = {
-    "VSR 300": {"fan_power": 166, "heater_power": 1670},
-    "VSR 500": {"fan_power": 338, "heater_power": 1670},
-    "VSR 150/B": {"fan_power": 74, "heater_power": 500},
-    "VTR 200/B (500Wt Heater)": {"fan_power": 168, "heater_power": 500},
-    "VTR 200/B (1000Wt Heater)": {"fan_power": 168, "heater_power": 1000},
+    "VSC 100": {"fan_power": 27, "heater_power": 0, "supply_fans": 1, "extract_fans": 0},
+    "VSC 200": {"fan_power": 81, "heater_power": 0, "supply_fans": 1, "extract_fans": 0},
+    "VSC 300": {"fan_power": 115, "heater_power": 0, "supply_fans": 1, "extract_fans": 0},
+    "VSR 150/B": {"fan_power": 70, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VSR 150/B L": {"fan_power": 70, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VSR 150/B R": {"fan_power": 70, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VSR 200/B L": {"fan_power": 162, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VSR 200/B R": {"fan_power": 162, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VSR 300": {"fan_power": 166, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VSR 400": {"fan_power": 338, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VSR 500": {"fan_power": 338, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VSR 700": {"fan_power": 340, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VTC 200 L": {"fan_power": 170, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 200 R": {"fan_power": 170, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 200-1 L": {"fan_power": 162, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 200-1 R": {"fan_power": 162, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 300 L": {"fan_power": 170, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 300 R": {"fan_power": 170, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 500 L": {"fan_power": 340, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 500 R": {"fan_power": 340, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 700 L": {"fan_power": 340, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTC 700 R": {"fan_power": 340, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTR 100/B": {"fan_power": 70, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/B L 500W": {"fan_power": 166, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/B L 1000W": {"fan_power": 166, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/B R 500W": {"fan_power": 166, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/B R 1000W": {"fan_power": 166, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/K L 500W": {"fan_power": 172, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/K L 1000W": {"fan_power": 172, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/K R 500W": {"fan_power": 172, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 150/K R 1000W": {"fan_power": 172, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VTR 250/B L 500W": {"fan_power": 162, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 250/B L 1000W": {"fan_power": 162, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VTR 250/B R 500W": {"fan_power": 162, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 250/B R 1000W": {"fan_power": 162, "heater_power": 1000, "supply_fans": 1, "extract_fans": 1},
+    "VTR 275/B L": {"fan_power": 162, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 275/B R": {"fan_power": 162, "heater_power": 500, "supply_fans": 1, "extract_fans": 1},
+    "VTR 300/B L": {"fan_power": 162, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VTR 300/B R": {"fan_power": 162, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VTR 350/B L": {"fan_power": 338, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VTR 350/B R": {"fan_power": 338, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VTR 500 L": {"fan_power": 340, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VTR 500 R": {"fan_power": 340, "heater_power": 1670, "supply_fans": 1, "extract_fans": 1},
+    "VTR 700 L": {"fan_power": 340, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
+    "VTR 700 R": {"fan_power": 340, "heater_power": 0, "supply_fans": 1, "extract_fans": 1},
 }
 
-
 # Constants from the old integration
 MAX_TEMP = 30
 MIN_TEMP = 12

custom_components/systemair/coordinator.py

@@ -41,6 +41,7 @@ def __init__(
         """Initialize."""
         self.client = client
         self.config_entry = config_entry
+
         super().__init__(
             hass=hass,
             logger=LOGGER,

custom_components/systemair/manifest.json

@@ -13,5 +13,5 @@
   "requirements": [
     "pymodbus>=3.11.1"
   ],
-  "version": "1.0.3"
-}
+  "version": "1.0.4"
+}

custom_components/systemair/sensor.py

@@ -223,7 +223,6 @@ class SystemairSensor(SystemairEntity, SensorEntity):
     """Systemair Sensor class."""
 
     _attr_has_entity_name = True
-
     entity_description: SystemairSensorEntityDescription
 
     def __init__(
@@ -280,6 +279,16 @@ def native_value(self) -> float | None:
         if not specs:
             return None
 
+        # Get fan counts from specs, defaulting to 0 if not present
+        num_supply_fans = specs.get("supply_fans", 0)
+        num_extract_fans = specs.get("extract_fans", 0)
+        total_fans = num_supply_fans + num_extract_fans
+
+        # Calculate power per fan
+        power_per_fan = 0
+        if total_fans > 0:
+            power_per_fan = specs.get("fan_power", 0) / total_fans
+
         # Get current fan speeds and heater status
         supply_fan_pct = self.coordinator.get_modbus_data(parameter_map["REG_OUTPUT_SAF"])
         extract_fan_pct = self.coordinator.get_modbus_data(parameter_map["REG_OUTPUT_EAF"])
@@ -288,13 +297,12 @@ def native_value(self) -> float | None:
         if supply_fan_pct is None or extract_fan_pct is None or heater_on is None:
             return None
 
-        # Assume max fan power is split 50/50 between supply and extract fans
-        max_fan_power_per_fan = specs["fan_power"] / 2
-
-        supply_power = (supply_fan_pct / 100) * max_fan_power_per_fan
-        extract_power = (extract_fan_pct / 100) * max_fan_power_per_fan
-        heater_power = specs["heater_power"] if heater_on else 0
+        # Calculate power for each component
+        supply_power = (supply_fan_pct / 100) * power_per_fan * num_supply_fans
+        extract_power = (extract_fan_pct / 100) * power_per_fan * num_extract_fans
+        heater_power = specs.get("heater_power", 0) if heater_on else 0
 
+        # Return the correct value based on the sensor's key
         key = self.entity_description.key
         if key == "supply_fan_power":
             return round(supply_power, 1)

custom_components/systemair/translations/en.json

@@ -2,17 +2,26 @@
     "config": {
         "step": {
             "user": {
-                "description": "You need enter IAM Module IP Address",
+                "title": "Connect to Systemair Unit",
+                "description": "Enter the connection details for your Systemair IAM module. Press Submit to check the connection.",
+                "data": {
+                    "host": "IP Address",
+                    "port": "Port",
+                    "slave_id": "Slave ID"
+                }
+            },
+            "model": {
+                "title": "Select Your Unit Model",
+                "description": "Choose your ventilation unit model. This is required for accurate power consumption calculations.",
                 "data": {
-                    "ip_address": "IP Address",
                     "model": "Ventilation Unit Model"
                 }
             }
         },
         "error": {
-            "connection": "Unable to connect to the server.",
-            "unknown": "Unknown error occurred.",
-            "already_configured": "This unit is already configured."
+            "cannot_connect": "Unable to connect to the device. Please check the IP address, port, and that the device is online.",
+            "unknown": "An unknown error occurred.",
+            "already_configured": "This device is already configured."
         }
     },
     "options": {