MickMake / GoSungrow

GoLang implementation to access the iSolarCloud API updated by SunGrow inverters.
https://mickmake.com/
GNU General Public License v2.0
185 stars 48 forks source link

Possible mismatched units #120

Open Paraphraser opened 2 months ago

Paraphraser commented 2 months ago

While writing a response to issue 117, I stumbled across the following:

Logger: homeassistant.components.sensor
Source: components/sensor/__init__.py:731
integration: Sensor (documentation, issues)
First occurred: August 30, 2024 at 11:35:55 PM (10 occurrences)
Last logged: August 30, 2024 at 11:35:56 PM

Entity sensor.gosungrow_virtual_9999999_p83013 (<class 'homeassistant.components.mqtt.sensor.MqttSensor'>) is using native unit of measurement 'Wh/㎡' which is not a valid unit for the device class ('irradiance') it is using; expected one of ['BTU/(h⋅ft²)', 'W/m²']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mqtt%22
Entity sensor.gosungrow_virtual_9999999_p83202 (<class 'homeassistant.components.mqtt.sensor.MqttSensor'>) is using native unit of measurement 'kWp' which is not a valid unit for the device class ('power') it is using; expected one of ['W', 'kW']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mqtt%22
Entity sensor.gosungrow_virtual_9999999_p83012 (<class 'homeassistant.components.mqtt.sensor.MqttSensor'>) is using native unit of measurement 'W/㎡' which is not a valid unit for the device class ('irradiance') it is using; expected one of ['BTU/(h⋅ft²)', 'W/m²']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mqtt%22
Entity sensor.gosungrow_getpsdetail_9999999_p83012 (<class 'homeassistant.components.mqtt.sensor.MqttSensor'>) is using native unit of measurement 'W/㎡' which is not a valid unit for the device class ('irradiance') it is using; expected one of ['BTU/(h⋅ft²)', 'W/m²']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mqtt%22
Entity sensor.gosungrow_getpsdetail_9999999_p83013 (<class 'homeassistant.components.mqtt.sensor.MqttSensor'>) is using native unit of measurement 'Wh/㎡' which is not a valid unit for the device class ('irradiance') it is using; expected one of ['BTU/(h⋅ft²)', 'W/m²']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mqtt%22

I don't recall ever before visiting:

Settings » System » Logs

so I have no idea whether these apparent unit mismatches are the result of recent changes to Home Assistant or have always been there. I also don't know whether these messages are attributable to GoSungrow using "incorrect units", or HA having "incorrect expectations" as to what the correct units should be. It just seems to me the existence of these mismatches should be documented so they can be addressed if/when time allows.