Implementing new Modbus-Registers for sungrow-batteries SBR

[Matze-aaa]

Hello, would it be possible to implement the new registers (new firmware SHxxRT from July 2023)?
https://www.photovoltaikforum.com/thread/166134-daten-lesen-vom-sungrow-wechselrichtern-modbus/?postID=3270446#post3270446

10711- 10720; Batt. Serial; S....
10721 - 10730; Batt FW; SBRBCU-S_22011.01.19
10741;Bat Spannung 0.1 V; 2010
10742;Bat Strom 0.1 A; 1
10743;Bat Temp 0.1 °C; 215
10744;SOC 0.1%; 1000
10745;SOH %, 99
10746-10747; in Batt gespeichert total kWh; 22751
10748-10749; aus Batt entnommen total kWh; 21501
10750;??; 1
10751;??; 4
10757;Max.Voltage of Cell 0.1mV; 33457 (also 3.3457 Volt)
10758;Position of Max.Voltage Cell; 518
10759;Min.Voltage of Cell 0.1mV; 33390
10760;Position of Min.Voltage Cell; 788
10761;Max. Temp. of Module 0.1 °C; 218
10762; Pos of Max. Temp. of Module ; 514
10763;Min. Temp. of Module 0.1 °C; 206
10764; Pos of Min. Temp. of Module ; 257
10765;Max. Cell Voltage of Module 1 0.1mV; 33440
10766;Max. Cell Voltage of Module 2 0.1mV; 33457
10767;Max. Cell Voltage of Module 4 0.1mV; 33423
10768, etc
10773;Min. Cell Voltage of Module 1 0.1mV; 33399
10774;Min. Cell Voltage of Module 2 0.1mV; 33398
10775;Min. Cell Voltage of Module 3 0.1mV; 33390
10768; etc
10781;Cell Type of Module 1; 1
10782;Cell Type of Module 2; 1
10783;Cell Type of Module 3; 1
10784;Cell Type of Module 4; 0
...
10789;Zustand DC-Schütz; 2

THX a lot in advance and for the great project!!

[dylan09]

These register are related to the Sungrow SBR battery unit. They are not from the inverter.
I think they should be in a separate Modbus configuration. You need Modbus slave ID 200 to get these registers from the inverter.

[Matze-aaa]

Ok, could s.o. start with an example for one or two registers? Then I could add the others via copy and paste 🙃😊👍🏻

[mkaiser]

I guess you can start by copying the modbus_sungrow.yaml file from this repo. Change the modbus ip / slave address and use trial / error :)

My battery is from pylontech, attached to a SH10.RT. So, I won't be able to test this.

[Matze-aaa]

Thx for the replies.
Solved.
I got an example from another user at PV-Forum, so I adjusted and integratet the lines in my

• modbus_sungrow.yaml (pasted at line 1726)
• secrets.yaml

2023-07-30_Batterieergänzung_modbus_sungrow.yaml.txt
2023-07-30_Secrets.yaml-mitBatterieErweiterung.txt

[mkaiser]

HA core 8.3 is /was not broken, it was the way I used the byte swap in the YAML that caused the issues.

@btota: Could you tell me what you paid for the battery? ;)

... did you remove the swap:word from the battery charge / discharge? This could explain the GWh energy values. The swap:none should only be applied (or the line just removed), if there is a single register read (count: 1)

[Matze-aaa]

thanks a lot for your advice.
Now I've seen your update.

So I've changed the battery-extension:
swap: word into swap: none (only in single register read, as you wrote).
Second I deleted these two blocks:

  - name: Battery 1 Serial Array
    unique_id: sg_battery_1_serial_array
    slave: !secret sungrow_modbus_slave_battery
    address: 10710 # reg 10711- 10720; Batt. Serial; S....
    input_type: input
    count: 10
    data_type: custom
    #structure: ">10H"  #https://docs.python.org/3/library/struct.html
    structure: ">20B"  #https://docs.python.org/3/library/struct.html
    scan_interval: 120

  - name: Battery 1 Firmware Array
    unique_id: sg_battery_1_firmware_array
    slave: !secret sungrow_modbus_slave_battery
    address: 10720 # reg 10721 - 10730; Batt FW; SBRBCU-S_22011.01.19
    input_type: input
    count: 10
    data_type: custom
    structure: ">20B"  #https://docs.python.org/3/library/struct.html
    scan_interval: 120

Now it works!

2023-08-22_modbus_sungrow-korrigiert_nur-Batterieerweiterung-SBR_ab1726.txt

[btota]

°_^ your right, GWh was a dream in a bubble :). changed --> works fine. As @Matze-aaa said block "Serial Array"+"Firmware Array" needs attention.

THX!

[Matze-aaa]

@btota
your advice to the changes in block "Serial Array"+"Firmware Array" i've read still now.
--> re-integrated it with your changes --> it works :-)
THX a lot!!

[Madcat71]

thank you for this, makes it more complete plugin/addon to HA for us who has the Sungrow battery aswell.

[Matze-aaa]

Following mkaiser's changes (in preparation for HA2023.10), I also changed the battery query extension:
removed all "count: 1/2";
removed all "swap: none"

2023-10-03_modbus_sungrow-korrigiert-nur-Batterieerweiterung_ab1533_NEU-fuer-HA2023.10.txt

[btota]

@Matze-aaa thx, for me, I had to comment both blocks "Array" (Serial, Firmware), I still have no clue which is the right "input_type" for these. Maybe someone with using registers in IObroker can help out?! Anyway, this is the way how it works for me ;).

[Matze-aaa]

@btota thx a lot! Indeed, I restored 2023.9 because it didn't work...
2023-10-08_modbus_sungrow-korrigiert-nur Batterieerweiterung-SBR_ab1533_NEU-fuer-HA2023.10.txt

[bjeanes]

I've found that these most of the battery registers (certainly all the module-specific ones) do not work if you are accessing it via WiNet-S (which expose the battery with Unit ID 2 instead of 200). Latest firmwares on WiNet-2, SBR128, and SH5.0RS

[Louisbertelsmann]

In this Integration it clearly states that the internal lan Port should be used, Else not all registers will Show up.

[bjeanes]

I acknowledge that. Not all inverters have an integrated LAN port (the SH5.0RS certainly does not, but I understand that generally the RSs do not, while the RTs do).

Hopefully the WiNet-S firmware will be updated to expose these registers. I can query these registers in the WiNet-S UI's "General Parameters" tab, and so can query them with libraries that talk to the WiNet-S over HTTP instead of Modbus, such as my own: https://github.com/bjeanes/modbus-mqtt/blob/main/sungrow-winets

[elektrinis]

I have the SBR battery (18 mo old), tried to read some battery registers (via integrated LAN port), but only zeros are returned. Mostly I'm interested in cell voltages.
FYI one of modules has failed recently and even sungrow support was unable to read the data, they sent me instructions how to use a voltmeter. So it looks like some models don't even report the data, or it is not implemented on receiving side in the inverter.
Was also unable to update battery firmware, as it does not appear as separate device in the list.

[mkaiser]

hi,

if the current version is working properly - could you make a Pullrequest and add it as "additional_sensors/modbus_sungrow_battery.yaml" to the repo? I guess it is better to have it in the repo than here in the discussions :)