Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Qcells ESS HYB-G3 not supported #104

Open
emichel11 opened this issue Jan 3, 2023 · 4 comments
Open

Qcells ESS HYB-G3 not supported #104

emichel11 opened this issue Jan 3, 2023 · 4 comments

Comments

@emichel11
Copy link

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)],

  •                vol.Length(min=200, max=200),
    
  •                vol.Length(min=200, max=300),
               )
           ),
           vol.Required("Information"): vol.Schema(
    
@MatthiasFraass
Copy link

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

@VadimKraus
Copy link
Contributor

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
Copy link

ifrh commented Oct 6, 2023

For the record:
The changes mentioned above in #104 (comment) seem to be the same as mentioned in #85 (comment)

@VadimKraus
Copy link
Contributor

VadimKraus commented Oct 29, 2023

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

Generally, based on #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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants