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
328 stars 86 forks source link

Differences SH*RT and SH*RS - battery capacity #17

Closed mkaiser closed 1 year ago

mkaiser commented 2 years ago

I can retrieve the battery capacity for my SH10.RT with this register

 - name: Battery Capacity 
        slave: !secret sungrow_modbus_slave
        address: 33048  #33049 
        input_type: holding
        count: 1
        data_type: uint16
        swap: word
        unit_of_measurement: kWh
        device_class: energy
        scale: 10
        scan_interval: 600

bjeanes reported, that it does not work on an SH5.0RS https://github.com/mkaiser/Sungrow-SHx-Inverter-Modbus-Home-Assistant/issues/13#issuecomment-1229277162

It would be nice to have more support for the RS (not RT)

Are there any more differences?

bjeanes commented 1 year ago

I think that's a better question for Sungrow as a company. Their Modbus implementation is all over the place and registers can work differently depending on the battery pack and firmware versions, as far as I've seen. Most notably for me, when my SH5.0RS arrived, battery_power was signed and was negative when charging, positive when discharging (despite docs showing it was _un_signed. A firmware upgrade changed that, which is kind of annoying actually, because having it signed was far more useful!

mkaiser commented 1 year ago

No progress here, closing....