[modbus sungrow] Expand add-on to include Running State, Power Flow Status and MPPT3/MPPT4 voltage/current

[owl770]

I am running V4.2.1 of the modbus sungrow add-on on a Raspberry Pi 3B+.

As per the attached document (Communication Protocol of Residential Hybrid Inverter-V1.1.4.pdf), could the modbus sungrow add-on be expanded to provide access to:

1. Running state - register 13000
2. Power flow status - register 13001
3. MPPT3 voltage - register 5015
4. MPPT3 current - register 5016
5. MPPT4 voltage - register 5115
6. MPPT4 current - register 5116

The changes are beyond my ability but I am able to assist with testing (I have a SH10RS inverter).

Communication Protocol of Residential Hybrid Inverter-V1.1.4.pdf

[owl770]

@soenkekueper - not sure if you are still involved in the sungrow modbus addon? Any assistance is greatly appreciated.

[soenkekueper]

Hey,
yes i'm still involved, but have little spare time at the moment.

As mentioned in the forum already (https://community.openhab.org/t/sungrow-binding/147442/99?u=soenke)
this is possible but needs a little refactoring. If one would just add these registers this would break the compatibility with inverters that don't provide this registers (as mine ;)).

So thanks for the issue, ill try to find some time - but can't give you a date for this.
Until then, you might just add a generic modbus poller thing for the missing registers.

BR Sönke

[owl770]

@soenkekueper - thanks for the info. Yes, I guess you would need to interrogate input register 5000 (Device type code) to determine which registers are valid and stop the addon making invalid queries.

Thanks for pointing me in the direction of the modbus poller thing. I can use that in the meantime.