Please add support for 3rd battery

[Thebasestar]

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

[jovana]

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

[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.

[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

[Thebasestar]

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]

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

[binsentsu]

If ready, you can make a pull request

[Thebasestar]

Done, right i hope