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
378 stars 95 forks source link

SH10RT-20 unknown inverter/running/system states, while exporting power from the battery for unknown reasons #365

Closed sisu70 closed 1 month ago

sisu70 commented 1 month ago

Is your feature request related to a problem? Please describe. I'm logging this as a features request rather than a bug, as it is referring to undocumented, afaik, values of some registers

Describe the solution you'd like Properly decode the three mentioned states and let the user understand the reason for the unexpected energy flow (battery to grid !)

Describe alternatives you've considered I tried to understand if the detected circumstances are documented, but it seems they aren't

Additional context Immagine 2024-10-09 165336

Referring to the time interval in the above graph when energy is exported to the grid from the battery, I found the following status values in HA's register:

(Template) sensor.sungrow_inverter_state = "Unknown - should not see me" (Modbus) sensor.running_state = 0x1D (Modbus) sensor.system_state = 0x8100

sisu70 commented 1 month ago

I now see that with the latest integration version

(Template) sensor.sungrow_inverter_state = "Unknown - should not see me" this should now report "De-rating Running"

(Modbus) sensor.system_state = 0x8100 because this value is now properly decoded

(Modbus) sensor.running_state = 0x1D but this is still not fully decoded, for what concerns bit 0x08 (but it may be always set)

At the end I think this "issue" can be closed and forgotten, as I still cannot understand why the detected behaviour can be classified as "derating", but it's not something related to the integration.

sisu70 commented 1 month ago

Closing