[solax] Support for X3 Ultra inverter (type=25)

[VasekCejka]

Please, could you try to implement support for X3 Ultra (type=25). There is already some support in
https://github.com/squishykid/solax/blob/master/solax/inverters/x3_ultra.py
I have tried to implement basic support in the layout of already implemented inverteds, see:
https://github.com/VasekCejka/openhab-addons/tree/Solax_new_X3Ultra_Inverter
This part was easy and seems to work (tested on separate OpenHAB 5.0.0 snapshot from date 2025-02-02). However there are some conceptional issues. This inverter supports 3rd PV line (PV3 Voltage, Current, Power), 2nd battery pack (Battery 2 Voltage, Current, Power, Remaining Capacity, Temperature) and EPS properties (EPS 1, 2, 3 Voltage, Current, Power, EPS Energy total, EPS Energy today), and some minor more. Support for these properties / channels is not implemented in the addon.
I'm able to perform testing, but I'm not experienced in development process, so I'm afraid not to break something.

Here I'm posting example of the raw data.
Retrieved content = {"sn":"SNXXXXXXX","ver":"1.003.11","type":25,"Data":[2366,2330,2365,124,107,121,2906,2472,2839,161,7260,7170,15,16,1114,1193,5009,5005,5004,2,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,4697,64526,60747,4698,65433,0,1,34,0,256,14594,268,6402,100,0,39,8003,0,0,7586,1,0,0,0,0,0,0,0,0,0,0,59,180,0,0,24691,0,23304,0,2,60,2710,1,121,2,0,0,2703,0,20700,8,1,0,1032,0,0,0,0,0,0,0,0,0,1,59,1,21,135,256,5256,3600,310,350,226,207,33,33,101,3268,3260,57271,73,0,0,0,0,0,0,0,0,7875,14,1122,0,0,0,0,0,0,0,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,60747,65535,59,8217,0,0,0,60698,65535,0,0,0,0,21302,14389,19269,12595,20531,12611,14640,0,0,0,0,0,0,0,0,0,512,2,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"Information":[30.000,25,"H3BC30K3065001",13,16.15,0.00,17.12,0.09,0.00,1]}
I'll post more when the sun will shine.
Thanx in advance!
Vasek

[openhab-bot]

This issue has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/solax-binding/146326/234

[theater]

I'll work on it when I have some free time.

[theater]

@VasekCejka I just had a brief look at your implementation...

However there are some conceptional issues. This inverter supports 3rd PV line (PV3 Voltage, Current, Power), 2nd battery pack (Battery 2 Voltage, Current, Power, Remaining Capacity, Temperature) and EPS properties (EPS 1, 2, 3 Voltage, Current, Power, EPS Energy total, EPS Energy today), and some minor more.

Not sure that I get what you mean but I would expect that these voltages/currents/power for the PV, EPS are probably like with the other 3-phase inverters, right? basically you have them for the three phases.
For the second battery pack I think we may need to introduce new channels if there are no existing ones...

[VasekCejka]

You are right, it is about new channels. Yes, it is 3-phase inverter.
Should we leave "Battery" channel withou number as Battery1 and add Battery2, or should we duplicate it? More ways are possible, but what would you as the author prefer?

[theater]

Wooh....I just see in the coding. Also it will be affected a lot because I always assumed to have only one battery :(
I have to think about it but so far very good job from your side. Checked your branch.