tolwi / hassio-ecoflow-cloud

EcoFlow Cloud Integration for Home Assistant
406 stars 74 forks source link

[BUG] Wrong measurement in Delta Pro #214

Open Just-Zuul opened 9 months ago

Just-Zuul commented 9 months ago

Hi, following Message i get in the Protokoll:

Logger: homeassistant.components.sensor Source: components/sensor/init.py:738 Integration: Sensor (documentation, issues) First occurred: 09:28:37 (3 occurrences) Last logged: 09:28:37

Entity sensor.02_speicher_1_re_main_design_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues Entity sensor.02_speicher_1_re_main_full_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues Entity sensor.02_speicher_1_re_main_remain_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues

knyghtryda commented 8 months ago

I am actually pretty surprised about this, but Home Assistant Core does not support mAh or Ah! I've added a discussion topic on home assistant core to hopefully add this: https://github.com/home-assistant/architecture/discussions/1052. I don't think error is affecting anything though, as the values still show correctly and are labeled correctly in the UI.

Just-Zuul commented 8 months ago

Well, I'm excited. Your arguments are correct regarding battery capacity. Let's see what happens....

CanD42 commented 3 months ago

Is there any update? I'm still getting these:

Entity sensor.delta_max_1_main_full_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues
Entity sensor.delta_max_1_main_remain_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues
Entity sensor.delta_max_2_main_design_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues
Entity sensor.delta_max_2_main_full_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues
Entity sensor.delta_max_2_main_remain_capacity (<class 'custom_components.ecoflow_cloud.sensor.CapacitySensorEntity'>) is using native unit of measurement 'mAh' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/tolwi/hassio-ecoflow-cloud/issues
Just-Zuul commented 3 months ago

No, its not. The Developer of HA means, that there are not enought Integrations what needs mAh so they don't want to integrate mAh.

But - its still work anyway - just the Protocol are still register the warning

Maybe at one niece sunny day.... ;-)