squishykid / solax

🌞 Solax Inverter API Wrapper
MIT License
100 stars 57 forks source link

Qcells ESS HYB-G3 not supported #104

Open emichel11 opened 1 year ago

emichel11 commented 1 year ago

I have a Qcells ESS HYB-G3 which is not supported. A query gives 300 elements curl -X POST http://192.123.456.789/ --data "optType=ReadRealTimeData&pwd=XXXXXXXXXXX" {"sn":"xxxxxxxxxx","ver":"3.003.02","type":14,"Data":[2290,2289,2297,0,0,0,0,0,0,0,0,0,0,0,0,0,5002,5001,5002,10,0,0,0,0,0,0,0,0,0,0,0,0,0,1,64901,65535,0,0,0,20760,0,0,2066,65535,65516,1,34,635,256,11547,4885,5644,100,0,21,0,0,0,0,0,0,0,0,0,0,0,0,0,12,0,1,0,0,0,10,0,0,0,0,0,3,0,1,0,0,0,0,0,769,0,0,0,689,0,0,0,0,0,0,0,0,0,1,28,1,21,17,256,2336,1600,130,234,148,136,32,32,1,1108,778,14649,14649,0,0,0,0,3235,3227,13474,0,20564,12339,18753,12341,18741,12868,12598,20564,12339,18754,12856,18738,13382,14648,20564,12339,18754,12856,18738,13638,13873,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1793,4097,1537,514,0,20760,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,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"Information":[6.000,14,"YYYYYYYYYYYYY",8,1.23,0.00,1.24,1.09,0.00,1]}%

There is a Qcell entry QCELLHYP3 but this defines only 200 elements. Following change works diff --git a/solax/inverters/qvolt_hyb_g3_3p.py b/solax/inverters/qvolt_hyb_g3_3p.py index f686d3e..44a320c 100644 --- a/solax/inverters/qvolt_hyb_g3_3p.py +++ b/solax/inverters/qvolt_hyb_g3_3p.py @@ -56,7 +56,7 @@ class QVOLTHYBG33P(Inverter): vol.Required("Data"): vol.Schema( vol.All( [vol.Coerce(float)],

MatthiasFraass commented 1 year ago

I can confirm that with your simple patch the Query works! Can you do a pull request so it gets integrated?

VadimKraus commented 1 year ago

this seems to be a newer firmware? The current sample was "ver": "2.034.06" and I can confirm it still works for my "Qcells ESS HYB-G3"

ifrh commented 11 months ago

For the record: The changes mentioned above in https://github.com/squishykid/solax/issues/104#issue-1517763728 seem to be the same as mentioned in https://github.com/squishykid/solax/issues/85#issuecomment-1228881287

VadimKraus commented 10 months ago

Ist this still an issue? Looking at the code, I guess it should be recognized as "X3 Hybrid G4".

Generally, based on https://github.com/squishykid/solax/issues/108 there seems to be an overlap without a clear distinction between Solax Power X3-Hybrid G4, Qcells Q.VOLT HYB-G3-3P and PEIMAR NOCTIS PSI-X3S.

Probably need merge and refactor the duplicates.