[solax] binding

Hey Konstantin,

the two channels are available now (with your V2 jar file) and they give correct values. Thank you, great work, ready for release!

Yes, I see them there, filled correctly. But I can’t access their channels to assign them to items:

String solaxSerialNumber "serialNumber [%s]"   { channel="solax:local-connect-inverter:localInverter:serialNumber"}

remains empty.
-Rupert

Thank you! I will move forward with the PR when the reviewer has time to do the review.
About the properties… initially I’ve supplied them also as channels but someone from the addon maintainers team asked me to remove it and to keep them only as properties. I think you cannot reference them anymore as channels.

Cheers,
K.

OK guys the PR has been merged so you can see it in the next SNAPSHOT release or you could wait for the next milestone release to use the “official” version :slight_smile:
Look out for the next milestone and the release notes for this change :slight_smile:

Thanks for the suggestions and the testing!

Hi Konstantin,

I think I found an error in V2: Channel total-consumption (32bit unsigned from fields 88 and 89) needs to be divided by 100. You probably programmed 10.

BTW, can I see the modified code somewhere?

Hi,
should be in the main branch of the official repo now when it’s merged.
https://github.com/openhab/openhab-addons/tree/main/bundles/org.openhab.binding.solax/src/test/java/org/openhab/binding/solax

Cheers,
Konstantin

I fixed that in this PR:

Sorry for chipping in, but I have exactly the same problem and as I’m a newbie to openhab, how would I manually create the item?

tia
regards

Welcome Dorfkind!

Which problem?

This is shown very well in the binding’s documentation Solax - Bindings | openHAB.

Hi Rupert, thanks for your answer.

Sorry for not defining my problem clearly enough. My problem is that Cloud_Connect_Inverter_Last_Retrieve_Time_Stamp just shows

yyyy-MM-dd HH:mm:ss and not the correct date, if I look at Administration/Model/…
However, it does show the correct day at Administration/Items, but not if I click on that value…thats pretty strange…hope its clearly enough?

Thanks for the link, I will work on that.

Regards
D

Hi Konstantin and all,
You did great job with this addon!
I’m new to this addon with Solax X3 Ultra (type=25) inverter. As I understand it, it is not supported for now, new parser should be made. Furtunately, there is already some work on it done on solax/solax/inverters/x3_ultra.py at master · squishykid/solax · GitHub .
Maybe I could try to implement support and test it. However there are some issues, I’m not sure about. For example there are Battery1 and Battery2 groups, EPS properties etc. For the first shot I can try to implement only channels defined in openhab-addons/bundles/org.openhab.binding.solax/src/main/java/org/openhab/binding/solax/internal/SolaxBindingConstants.java at main · openhab/openhab-addons · GitHub . I’m not much used to java programming, neither to addon development or FVE experience, so I’d like to ask for help and patience. Or is there any work done on this subject?
As an example I’m posting 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]}

Vasek

Hi Vasek,
I’ll give it a try. Could you please create a new issue in github with the information above and assign it to my name.
I’ll try to implement a parser but the testing will go to you. :slight_smile:
Also would be great if you could provide (like you did above) the retrieved content and the expected values.
I usually implement first a unit test for the parser which tries to retrieve the data from the raw content and I cross-check the expected values (for example input current = 20A, consumption at the moment = 4500W, etc). After that the integration becomes piece of cake :slight_smile:

Cheers,
Konstantin

Thanx! I have created an issue:

But I do not know, how to assign it to you. Probably I do not have write access to the repository.
Vasek