Remove fan_status entity (v0.2.10)

Author: Ted Roberts

custom_components/novastar_h/api.py

@@ -59,7 +59,6 @@ class NovastarState:
     temp_status: int | None = None  # Temperature status (0=normal, 1=warning, etc.)
     device_status: int | None = None  # Device status (0=busy, 1=ready)
     signal_status: int | None = None  # Signal power status from iSignal
-    fan_status: list[int] = field(default_factory=list)  # Fan status list from fanList
     ftb_active: bool = False  # Fade to black (blackout) active
     freeze_active: bool = False  # Screen freeze active
     current_preset_id: int = -1  # -1 means no preset active
@@ -412,7 +411,6 @@ async def async_get_state(
         state.temp_status = temp_data.get("temp_status")
         state.device_status = temp_data.get("device_status")
         state.signal_status = temp_data.get("signal_status")
-        state.fan_status = temp_data.get("fan_status") or []
 
         return state
 
@@ -426,15 +424,13 @@ async def async_get_device_status_info(
         - temp_status: Temperature status code
         - device_status: Device status (0=busy, 1=ready)
         - signal_status: Signal power status from iSignal
-        - fan_status: List of fan status values from fanList
         """
         data = await self._async_request("device/readDetail", {"deviceId": device_id})
         result: dict[str, Any] = {
             "temperature": None,
             "temp_status": None,
             "device_status": None,
             "signal_status": None,
-            "fan_status": [],
         }
         if data and isinstance(data, dict):
             temp = data.get("backboardTemperature")
@@ -449,18 +445,9 @@ async def async_get_device_status_info(
             signal_status = data.get("iSignal")
             if signal_status is not None and isinstance(signal_status, (int, float)):
                 result["signal_status"] = int(signal_status)
-            fan_list = data.get("fanList")
-            if fan_list is not None and isinstance(fan_list, list):
-                result["fan_status"] = [int(f) for f in fan_list if isinstance(f, (int, float))]
         return result
 
-    async def async_get_temperature_info(
-        self, device_id: int = 0
-    ) -> dict[str, Any]:
-        """Get device temperature info (deprecated, use async_get_device_status_info)."""
-        return await self.async_get_device_status_info(device_id)
-
     async def async_get_temperature(self, device_id: int = 0) -> float | None:
         """Get device backboard temperature in Celsius."""
-        info = await self.async_get_temperature_info(device_id)
+        info = await self.async_get_device_status_info(device_id)
         return info.get("temperature")

custom_components/novastar_h/manifest.json

@@ -21,5 +21,5 @@
     }
   ],
   "zeroconf": ["_novastar._tcp.local."],
-  "version": "0.2.9"
+  "version": "0.2.10"
 }

custom_components/novastar_h/sensor.py

@@ -1,7 +1,5 @@
 from __future__ import annotations
 
-from typing import Any
-
 from homeassistant.components.sensor import (
     SensorDeviceClass,
     SensorEntity,
@@ -32,7 +30,6 @@ async def async_setup_entry(
         NovastarTempStatusSensor(entry, coordinator, device_info),
         NovastarDeviceStatusSensor(entry, coordinator, device_info),
         NovastarSignalStatusSensor(entry, coordinator, device_info),
-        NovastarFanStatusSensor(entry, coordinator, device_info),
     ])
 
 
@@ -255,78 +252,3 @@ def native_value(self) -> str | None:
         return None
 
 
-class NovastarFanStatusSensor(CoordinatorEntity[NovastarCoordinator], SensorEntity):
-    """Sensor entity for fan status."""
-
-    _attr_has_entity_name = True
-    _attr_name = "Fan Status"
-    _attr_translation_key = "fan_status"
-
-    # Map status codes to human-readable values
-    FAN_STATUS_MAP = {
-        0: "Off",
-        1: "Running",
-        2: "Error",
-    }
-
-    def __init__(
-        self,
-        entry: ConfigEntry,
-        coordinator: NovastarCoordinator,
-        device_info: NovastarDeviceInfo,
-    ) -> None:
-        """Initialize the sensor entity."""
-        super().__init__(coordinator)
-        self._entry = entry
-        self._device_info = device_info
-        self._attr_unique_id = f"{entry.entry_id}_fan_status"
-
-    @property
-    def device_info(self):
-        """Return device info."""
-        model = "H Series"
-        if self._device_info.model_id:
-            model = f"H Series (Model {self._device_info.model_id})"
-        return {
-            "identifiers": {(DOMAIN, self._entry.entry_id)},
-            "manufacturer": "Novastar",
-            "model": model,
-            "name": self._entry.data.get(CONF_NAME, DEFAULT_NAME),
-            "sw_version": self._device_info.firmware,
-            "serial_number": self._device_info.serial,
-        }
-
-    @property
-    def available(self) -> bool:
-        """Return True if entity is available."""
-        return self.coordinator.last_update_success
-
-    @property
-    def native_value(self) -> str | None:
-        """Return current fan status summary."""
-        if self.coordinator.data and self.coordinator.data.fan_status:
-            fan_list = self.coordinator.data.fan_status
-            # Count fans by status
-            running = sum(1 for f in fan_list if f == 1)
-            errors = sum(1 for f in fan_list if f == 2)
-            total = len(fan_list)
-            
-            if errors > 0:
-                return f"{errors}/{total} Error"
-            if running == total:
-                return "All Running"
-            if running == 0:
-                return "All Off"
-            return f"{running}/{total} Running"
-        return None
-
-    @property
-    def extra_state_attributes(self) -> dict[str, Any] | None:
-        """Return detailed fan status as attributes."""
-        if self.coordinator.data and self.coordinator.data.fan_status:
-            fan_list = self.coordinator.data.fan_status
-            attrs = {}
-            for i, status in enumerate(fan_list):
-                attrs[f"fan_{i + 1}"] = self.FAN_STATUS_MAP.get(status, f"Unknown ({status})")
-            return attrs
-        return None

custom_components/novastar_h/strings.json

@@ -116,9 +116,6 @@
       },
       "signal_status": {
         "name": "Signal Power Status"
-      },
-      "fan_status": {
-        "name": "Fan Status"
       }
     }
   }

custom_components/novastar_h/translations/en.json

@@ -116,9 +116,6 @@
       },
       "signal_status": {
         "name": "Signal Power Status"
-      },
-      "fan_status": {
-        "name": "Fan Status"
       }
     }
   }