home-assistant-solaredge-modbus icon indicating copy to clipboard operation
home-assistant-solaredge-modbus copied to clipboard

Please add support for 3rd battery

Open Thebasestar opened this issue 1 year ago • 7 comments

Hi, can you please add support for 3rd battery? i own a setup with se6000h+3x se batterybank and i have no way to setup energy dashboard on HA cause of missing data from 3rd battery (calculated sensor etc ) My setup is all modbus connected (no energynet) and i can see values of all 3 batteries on SE cloud monitoring platform. (read somewhere 3rd battery datas are not available from modbus)

Many thanks for your great job

Thebasestar avatar Mar 19 '23 15:03 Thebasestar

You can check this more up to date repo, probably it has already support for the 3rd battery: https://github.com/WillCodeForCats/solaredge-modbus-multi/discussions/296

jovana avatar May 18 '23 12:05 jovana

Nope, unfortunately this is a limitation of the (published) modbus register specification from Solaredge. There is no third battery in the specification and therefore it is impossible to support.

mpredfearn avatar May 18 '23 13:05 mpredfearn

Nope, unfortunately this is a limitation of the (published) modbus register specification from Solaredge. There is no third battery in the specification and therefore it is impossible to support.

The 3rd battery has an offset of +0x0200 on top of battery 2. I.e. Battery 3 State of Energy = 0xe484

regardts-hub avatar Jan 04 '24 04:01 regardts-hub

Nope, unfortunately this is a limitation of the (published) modbus register specification from Solaredge. There is no third battery in the specification and therefore it is impossible to support.

The 3rd battery has an offset of +0x0200 on top of battery 2. I.e. Battery 3 State of Energy = 0xe484

Great, you made my day!!!!! Lets hope binsetsu implement it soon. Thanks again

Thebasestar avatar Jan 04 '24 08:01 Thebasestar

Well for everybody needings, i made this modification by myself, and it work well adding battery 3 values to homeassistant. I'm not an expert of github, i ve just made copy paste adding strings missing for 3rd battery, but i'd like to share to all if possible, @binsentsu i'm here if you want.

I made a fork , only en lang, hope it work

Thebasestar avatar Jan 20 '24 11:01 Thebasestar

If ready, you can make a pull request

binsentsu avatar Jan 20 '24 15:01 binsentsu

Done, right i hope

Thebasestar avatar Jan 20 '24 15:01 Thebasestar