search

MatterVN / ModbusTCP2MQTT

Sungrow & SMA Solar Inverter addon for Home Assistant using mobusTCP
MIT License
68 stars 50 forks source link

sensor.inverter_daily_energy_import off by a factor of 1000 #15

Open nvx opened 2 years ago

nvx commented 2 years ago

sensor.inverter_daily_energy_import claims to be kWh unit_of_measurement, but comparing the values reported vs the values in the app it looks to be smaller by a factor of 1000.

Eg the app shows 4.9 kWh, while the addon is reporting 0.0049 kWh

nvx commented 2 years ago

This appears to have been broken by 2.1.1 https://github.com/TenySmart/ModbusTCP2MQTT/commit/b89fd38d45c136f95d7c7497152ba1d6d46076f3#diff-e105e110db1d93e59ab312f31961e5e96944ea056f4cfd2097e07d5b28373d53R317 where a divide by 1000 was added.

Perhaps some models report this in kWh already while other models report it in Wh? I'm on a sg5kd