yearly value is too high

mkaiser / Sungrow-SHx-Inverter-Modbus-Home-Assistant

Sungrow SH Integration for Home Assistant for SH3K6, SH4K6, SH5K-20, SH5K-V13, SH3K6-30, SH4K6-30, SH5K-30, SH3.RS, SH3.6RS, SH4.0RS, SH5.0RS, SH6.0RS, SH5.0RT, SH6.0RT, SH8.0RT, SH10RT, SH5.0RT-20, SH6.0RT-20, SH8.0RT-20, SH10RT-20, SH5.0RT-V112, SH6.0RT-V112, SH8.0RT-V112, SH10RT-V112, SH5.0RT-V122, SH6.0RT-V122, SH8.0RT-V122, SH10RT-V122, SH4.6R

361 stars 94 forks source link

yearly value is too high #363

Open svenschedlbauer opened 4 weeks ago

svenschedlbauer commented 4 weeks ago

Describe the bug: The current yearly pv generation is 4.400 kwh, but sensor.yearly_pv_generation_current is showing 12.254 kwh. The monthly pv generation is showing the correct value.

Your Sungrow inverter:

Model: [e.g. [SH6.0RT]
The inverter is connected via (mark one)
- [] LAN (internal port)

Home Assistant version:

Version: 2024.10

modbus_sungrow.yaml:

Version/ time stamp : 2024-10-01

Gnarfoz commented 3 weeks ago

The yearly registers showing wrong values is sadly a known firmware issue. You can try updating, but as far as I know, there is no fixed version. This integration should probably switch to providing a template sensor that aggregates the monthly values instead.

FJJT01 commented 3 weeks ago

Same for the sensor.yearly_export_current and the specific year sensors (at least sensor.yearly_pv_generation_2024 and sensor.yearly_export_2024).