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
342 stars 92 forks source link

SH5k-30 issues #241

Open TaylorWoods1 opened 8 months ago

TaylorWoods1 commented 8 months ago

Hi All,

seeking some help on integration with SH5K-30 inverter. it seems that writing parameters is ineffective. written parameters remain from anywhere between a few seconds, and i have even seen up to a day on some markers, however they are eventually overwritten once again by the inverter. examples are: battery_capacity = 655.35 kWh ems_mode_selection = self-consumption mode (default) battery_forced_charge_discharge_cmd = stop (default)

no matter what i do, these will always end up back at these values after a period of time.

Im assuming based on this, that the integration with SH5k-30 is not great and writing parameters is useless? am i missing something?

second issue is that the battery SOC on the inverter is so unrealistic and it is forcing a CV charge state when the inverter thinks the battery is approaching the max battery voltage, yet my BMS is reporting a completely different SOC value. is there a way to force sync these on HA?

btota commented 8 months ago

battery_capacity = 655.35 kWh

seems register adress is wrong 13038 (13039 -1)

do you use newest *.yaml from mkaiser? HA uptodate? modbus-proxy? can you check your modbus_sungrow.yaml, beneath you'll find my conf for Sungrow SH10RT-V112 with SBR 128


- name: Battery capacity
        unique_id: sg_battery_capacity
        device_address: !secret sungrow_modbus_slave
        address: 13038 # reg 13039
        #address: 33048 # reg 33049
        input_type: input
        #input_type: holding
        data_type: uint16
        unit_of_measurement: kWh
        device_class: energy_storage
        scale: 0.1
        scan_interval: 600
TaylorWoods1 commented 8 months ago

thanks for the response @btota. im using mkaiser's latest version from 3 weeks ago, dated 31-12-23. home assistant is Core 2024.1.3, Supervisor 2023.12.1, Operating System 11.4, Frontend 20240104.0

  - name: Battery capacity
    unique_id: sg_battery_capacity
    device_address: !secret sungrow_modbus_slave
    address: 33048 # reg 33049
    input_type: holding
    data_type: uint16
    unit_of_measurement: kWh
    device_class: energy_storage
    scale: 0.01
    scan_interval: 600
TaylorWoods1 commented 8 months ago

sungrow_modbus_host_ip: 192.168.86.61 sungrow_modbus_port: 502 sungrow_modbus_slave: 1

TaylorWoods1 commented 8 months ago

@btota where are you getting your register addresses from?

btota commented 8 months ago

TI_20231019_Communication.Protocol.of.Residential.Hybrid.Inverter_V1.1.2_EN.pdf

github-actions[bot] commented 3 days ago

This issue is stale because it has been open for 30 days with no activity.