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

Update inverterData.js #197

Merged
merged 1 commit into from
Feb 4, 2024
Merged

Update inverterData.js #197

merged 1 commit into from
Feb 4, 2024

Conversation

Andre-0815-4711
Copy link
Contributor

Moin,

ich habe heute mal einige Werte die unklar waren überprüft. Zeile 57: der Fehler "State value to set for "solax.3.info.version" has to be type "string" but received type "number", das ist offensichtlich die Softwareversion der Wallbox gleich zu setzen mit der DSP/ARM Version des X3-Hybrid (Anzeige in der APP). Zeile 272: geändert in Charger Status und die Stati passe ich auch gleich noch in der Main.js an. Die Werte/Beschreibung habe ich 1:1 aus der Solax App übernommen. Zeile 289: der Wert 22 gridacpower stellt mich aktuell noch vor ein Rätsel. In der APP wird er mit Netzleistung angegeben aber wenn der Akku vom Haus geladen wird zeigt er diesen Wert (wenn negativ) an. Erst mal weg lassen. Zeile 291: Wert 26, das ist der Status des Steckers, müsste identisch wie data.chargerstatus und data.chargemode behandelt werden, 0=Not Connected 1=Connected 2=Charging Zeile 292: Wert 80, das ist definitiv die Dauer des aktuellen Ladevorgangs in Sekunden. wenn man den nicht direkt in hh:mm:ss umrechnen/anzeigen lassen kann würde ich den so lassen.

MfG
André

Moin,

ich habe heute mal einige Werte die unklar waren überprüft.
Zeile 57: der Fehler "State value to set for "solax.3.info.version" has to be type "string" but received type "number", das ist offensichtlich die Softwareversion der Wallbox gleich zu setzen mit der DSP/ARM Version des X3-Hybrid (Anzeige in der APP).
Zeile 272: geändert in Charger Status und die Stati passe ich auch gleich noch in der Main.js an. Die Werte/Beschreibung habe ich 1:1 aus der Solax App übernommen.
Zeile 289: der Wert 22 gridacpower stellt mich aktuell noch vor ein Rätsel. In der APP wird er mit Netzleistung angegeben aber wenn der Akku vom Haus geladen wird zeigt er diesen Wert (wenn negativ) an. Erst mal weg lassen.
Zeile 291: Wert 26, das ist der Status des Steckers, müsste identisch wie data.chargerstatus und data.chargemode behandelt werden, 0=Not Connected 1=Connected 2=Charging
Zeile 292: Wert 80, das ist definitiv die Dauer des aktuellen Ladevorgangs in Sekunden. wenn man den nicht direkt in hh:mm:ss umrechnen/anzeigen lassen kann würde ich den so lassen.

MfG
André
@simatec
Copy link
Owner

simatec commented Jan 26, 2024

Umbenennen der States ist im Nachgang echt Mist... Hier gibt es Konflikte bi Updates der Usersysteme, da die alten States alle als Datenleichen liegen bleiben. Im Nachgang ärgere ich mich sehr, dass ich deine Daten, die unvollständig und fehlerhaft waren, überhaupt aufgenommen habe... Nun gibt es ein riesen Durcheinander und ein haufen PR's

@Andre-0815-4711
Copy link
Contributor Author

Moin,

ja das ist Mist, da stimme ich Dir voll und ganz zu.
Mit der "gridacpower" hat mich Solax auch verarscht, der Wert wird so "falsch beschriftet" in der APP dargestellt.
image
Auf dem Screenshot sind die -240 Watt auch wirklich der aktuelle Netzbedarf vom Haus (Akku ist alle und PV noch nicht aktiv). Das passt ja auch mit der Netzleistung der einzelnen Phasen darüber.
Wenn dann aber die Grundlast vom Haus durch PV gedeckt ist und der Akku geladen wird, geht der Wert ins positive und ist die Leistung mit der der Akku geladen wird. Ich nehme an, dass Solax mit diesem Wert steuert wie viel Leistung aus dem Netz zum laden des Autos verwendet werden soll. Im X3-Hybrid habe ich eingestellt, dass das Akku vom Haus nie zum laden des Autos verwendet werden soll.

Bis auf die falsche Definition des Daten Typ der Versions sind die restlichen "Schnitzer" aber nicht von mir ;-).

Für zukünftige Änderungsvorschläge nehme ich mit:

  1. ID der Datenpunkte nur dann in den PullRequest wenn absolut Sicher feststeht was das für ein Wert ist, andern Falls nur Beschreiben wie ich den Wert gefunden habe und was es sein könnte. dann kann man sich darüber austauschen.
  2. Ist es auch ein Problem, wenn der Daten Typ bei der Erstanlage falsch ist oder wird der bei einer Änderung überschrieben?
  3. bei Sachen im Adapter die ich nicht verstehe, lasse ich sowieso die Hände von...

Im Screenshot unter Strom die 3 Werte -0.7A, 0.4A und 0.6A habe ich auch schon eindeutig zuordnen können, aber hier ist der Name auch völlig unklar, egal welcher E-Auto Ladestatus, Hauslast oder Wechselricher-Last, die Werte bleiben fast unverändert im 0, Bereich mal negativ, mal positiv.

Wäre es denn potentiell auch möglich mit dem Adapter Werte in der Wallbox oder den Wechselrichtern zu schreiben/setzen?
Der Modbus zwischen Wechselrichter und Wallbox ist mit deren Daten derart blockiert, dass man da keine Chance hat mit rein zu funken.
Von meinem aktuellen Verständnis müsste das aber auch nur über LAN funktionieren? Oder hat die SolaxCloud App eine eigene Modbus TCP Komponente?

MfG
André

@Andre-0815-4711
Copy link
Contributor Author

Ich habe mir die Änderungen aus dem request noch mal genauer angeschaut um den Ärger über die Datenleichen zu reduzieren .
data.plugged und data.chargeduration brauchen nicht zwingend umbenannt werden, da reicht die Anpassung der Description. Bei data.plugged musste ja der Lockup hinzugefügt werden und bei data.chargeduration war aufgrund der Beschreibung nur der Datentyp falsch.

@simatec simatec merged commit 0143230 into simatec:master Feb 4, 2024
11 checks passed
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

Successfully merging this pull request may close these issues.

2 participants