From c45675a01fe5a5f0689455b3211d36ba56ad7fcb Mon Sep 17 00:00:00 2001 From: Dave T <17680170+davet2001@users.noreply.github.com> Date: Tue, 3 Mar 2026 20:34:44 +0000 Subject: [PATCH] Add additional diagnostic sensors to aurora_abb_powerone PV inverter (#164622) --- .../aurora_abb_powerone/coordinator.py | 12 ++++ .../components/aurora_abb_powerone/sensor.py | 55 +++++++++++++++++++ .../aurora_abb_powerone/strings.json | 21 +++++++ .../aurora_abb_powerone/test_sensor.py | 14 ++++- 4 files changed, 101 insertions(+), 1 deletion(-) diff --git a/homeassistant/components/aurora_abb_powerone/coordinator.py b/homeassistant/components/aurora_abb_powerone/coordinator.py index d38f0716b44..64859ddc372 100644 --- a/homeassistant/components/aurora_abb_powerone/coordinator.py +++ b/homeassistant/components/aurora_abb_powerone/coordinator.py @@ -61,7 +61,13 @@ class AuroraAbbDataUpdateCoordinator(DataUpdateCoordinator[dict[str, float]]): frequency = self.client.measure(4) i_leak_dcdc = self.client.measure(6) i_leak_inverter = self.client.measure(7) + power_in_1 = self.client.measure(8) + power_in_2 = self.client.measure(9) temperature_c = self.client.measure(21) + voltage_in_1 = self.client.measure(23) + current_in_1 = self.client.measure(25) + voltage_in_2 = self.client.measure(26) + current_in_2 = self.client.measure(27) r_iso = self.client.measure(30) energy_wh = self.client.cumulated_energy(5) [alarm, *_] = self.client.alarms() @@ -87,7 +93,13 @@ class AuroraAbbDataUpdateCoordinator(DataUpdateCoordinator[dict[str, float]]): data["grid_frequency"] = round(frequency, 1) data["i_leak_dcdc"] = i_leak_dcdc data["i_leak_inverter"] = i_leak_inverter + data["power_in_1"] = round(power_in_1, 1) + data["power_in_2"] = round(power_in_2, 1) data["temp"] = round(temperature_c, 1) + data["voltage_in_1"] = round(voltage_in_1, 1) + data["current_in_1"] = round(current_in_1, 1) + data["voltage_in_2"] = round(voltage_in_2, 1) + data["current_in_2"] = round(current_in_2, 1) data["r_iso"] = r_iso data["totalenergy"] = round(energy_wh / 1000, 2) data["alarm"] = alarm diff --git a/homeassistant/components/aurora_abb_powerone/sensor.py b/homeassistant/components/aurora_abb_powerone/sensor.py index d35d8a2d8cb..fdc9172bba6 100644 --- a/homeassistant/components/aurora_abb_powerone/sensor.py +++ b/homeassistant/components/aurora_abb_powerone/sensor.py @@ -68,6 +68,7 @@ SENSOR_TYPES = [ entity_category=EntityCategory.DIAGNOSTIC, native_unit_of_measurement=UnitOfFrequency.HERTZ, state_class=SensorStateClass.MEASUREMENT, + translation_key="grid_frequency", entity_registry_enabled_default=False, ), SensorEntityDescription( @@ -88,6 +89,60 @@ SENSOR_TYPES = [ translation_key="i_leak_inverter", entity_registry_enabled_default=False, ), + SensorEntityDescription( + key="power_in_1", + device_class=SensorDeviceClass.POWER, + entity_category=EntityCategory.DIAGNOSTIC, + native_unit_of_measurement=UnitOfPower.WATT, + state_class=SensorStateClass.MEASUREMENT, + translation_key="power_in_1", + entity_registry_enabled_default=False, + ), + SensorEntityDescription( + key="power_in_2", + device_class=SensorDeviceClass.POWER, + entity_category=EntityCategory.DIAGNOSTIC, + native_unit_of_measurement=UnitOfPower.WATT, + state_class=SensorStateClass.MEASUREMENT, + translation_key="power_in_2", + entity_registry_enabled_default=False, + ), + SensorEntityDescription( + key="voltage_in_1", + device_class=SensorDeviceClass.VOLTAGE, + entity_category=EntityCategory.DIAGNOSTIC, + native_unit_of_measurement=UnitOfElectricPotential.VOLT, + state_class=SensorStateClass.MEASUREMENT, + translation_key="voltage_in_1", + entity_registry_enabled_default=False, + ), + SensorEntityDescription( + key="current_in_1", + device_class=SensorDeviceClass.CURRENT, + entity_category=EntityCategory.DIAGNOSTIC, + native_unit_of_measurement=UnitOfElectricCurrent.AMPERE, + state_class=SensorStateClass.MEASUREMENT, + translation_key="current_in_1", + entity_registry_enabled_default=False, + ), + SensorEntityDescription( + key="voltage_in_2", + device_class=SensorDeviceClass.VOLTAGE, + entity_category=EntityCategory.DIAGNOSTIC, + native_unit_of_measurement=UnitOfElectricPotential.VOLT, + state_class=SensorStateClass.MEASUREMENT, + translation_key="voltage_in_2", + entity_registry_enabled_default=False, + ), + SensorEntityDescription( + key="current_in_2", + device_class=SensorDeviceClass.CURRENT, + entity_category=EntityCategory.DIAGNOSTIC, + native_unit_of_measurement=UnitOfElectricCurrent.AMPERE, + state_class=SensorStateClass.MEASUREMENT, + translation_key="current_in_2", + entity_registry_enabled_default=False, + ), SensorEntityDescription( key="alarm", device_class=SensorDeviceClass.ENUM, diff --git a/homeassistant/components/aurora_abb_powerone/strings.json b/homeassistant/components/aurora_abb_powerone/strings.json index 0a0b43dba91..4b65177b4bf 100644 --- a/homeassistant/components/aurora_abb_powerone/strings.json +++ b/homeassistant/components/aurora_abb_powerone/strings.json @@ -24,9 +24,18 @@ "alarm": { "name": "Alarm status" }, + "current_in_1": { + "name": "String 1 current" + }, + "current_in_2": { + "name": "String 2 current" + }, "grid_current": { "name": "Grid current" }, + "grid_frequency": { + "name": "Grid frequency" + }, "grid_voltage": { "name": "Grid voltage" }, @@ -36,6 +45,12 @@ "i_leak_inverter": { "name": "Inverter leak current" }, + "power_in_1": { + "name": "String 1 power" + }, + "power_in_2": { + "name": "String 2 power" + }, "power_output": { "name": "Power output" }, @@ -44,6 +59,12 @@ }, "total_energy": { "name": "Total energy" + }, + "voltage_in_1": { + "name": "String 1 voltage" + }, + "voltage_in_2": { + "name": "String 2 voltage" } } } diff --git a/tests/components/aurora_abb_powerone/test_sensor.py b/tests/components/aurora_abb_powerone/test_sensor.py index 0de8d923bb8..39587aa4103 100644 --- a/tests/components/aurora_abb_powerone/test_sensor.py +++ b/tests/components/aurora_abb_powerone/test_sensor.py @@ -38,7 +38,13 @@ def _simulated_returns(index, global_measure=None): 4: 50.789, # frequency 6: 1.2345, # leak dcdc 7: 2.3456, # leak inverter + 8: 12.345, # power in 1 + 9: 23.456, # power in 2 21: 9.876, # temperature + 23: 123.456, # voltage in 1 + 25: 0.9876, # current in 1 + 26: 234.567, # voltage in 2 + 27: 1.234, # current in 2 30: 0.1234, # Isolation resistance 5: 12345, # energy } @@ -116,9 +122,15 @@ async def test_sensors(hass: HomeAssistant, entity_registry: EntityRegistry) -> sensors = [ ("sensor.mydevicename_grid_voltage", "235.9"), ("sensor.mydevicename_grid_current", "2.8"), - ("sensor.mydevicename_frequency", "50.8"), + ("sensor.mydevicename_grid_frequency", "50.8"), ("sensor.mydevicename_dc_dc_leak_current", "1.2345"), ("sensor.mydevicename_inverter_leak_current", "2.3456"), + ("sensor.mydevicename_string_1_power", "12.3"), + ("sensor.mydevicename_string_2_power", "23.5"), + ("sensor.mydevicename_string_1_voltage", "123.5"), + ("sensor.mydevicename_string_1_current", "1.0"), + ("sensor.mydevicename_string_2_voltage", "234.6"), + ("sensor.mydevicename_string_2_current", "1.2"), ("sensor.mydevicename_isolation_resistance", "0.1234"), ] for entity_id, _ in sensors: