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

[fronius-solarapi-v1] add currents #16198

Closed
wants to merge 7 commits into from
Closed

Conversation

mdkeil
Copy link
Contributor

@mdkeil mdkeil commented Sep 18, 2024

add currents for load management

kann ich leider bei mir nicht testen, da ich aktuell kein build-environment aktiv habe.. zudem muss dafür zwingend ein Smart-Meter verbaut sein.. ggfs. müsste man dies zur Sicherheit noch durch einen zusätzlichen Parameter abfangen.

Es ist zumindest eine Idee, um #16147 aufzulösen, obgleich dies auch über modbus gehen müsste.

mdkeil and others added 2 commits September 18, 2024 12:46
add currents for load management
add currents for load management
@andig
Copy link
Member

andig commented Sep 18, 2024

Wichtig wäre jetzt zu wissen, ob das immer und bei allen potenziellen Zählern verfügbar wäre?

@andig andig added the devices Specific device support label Sep 18, 2024
@mdkeil
Copy link
Contributor Author

mdkeil commented Sep 18, 2024

irgendwie habe ich mich in meiner build-umgebung vertan.. bitte die letzen beiden commits droppen/ignorieren. . Ich benutze das einfach zu wenig..

Wichtig wäre jetzt zu wissen, ob das immer und bei allen potenziellen Zählern verfügbar wäre?

Das kann ich so pauschal nicht beantworten.

@mdkeil
Copy link
Contributor Author

mdkeil commented Sep 18, 2024

Was ich schonmal sagen kann, das die Änderung bei mir funktioniert und Ströme angezeigt werden.. nachdem ich irgendwie meine VSCode-Umgebung wieder zum Laufen gebracht habe.. Meiner Ansicht nach müssen aber alle, die grid nutzen auch ein entsprechendes Smartmeter angebunden haben.. kann ich aber nicht 100% sagen.

@andig andig marked this pull request as draft September 18, 2024 19:16
@andig
Copy link
Member

andig commented Sep 18, 2024

Könnte man das bei Fronius erfragen? Für den WR reicht Leistung zur Steuerung und dann macht ein Merge bei allen Anwendern ohne Ströme im Zähler Fehler...

@mdkeil
Copy link
Contributor Author

mdkeil commented Sep 18, 2024

Was man machen kann, einen zusätzlichen Parameter einzubauen, der die Strommessung aktiviert.. default auf false und jeder der die Stromwerte z.B. für das Lastmanagement benötigt, kann dies manuell aktivieren. Noch eine kurzer Hilfe-Text und fertig.

@benesolar
Copy link
Contributor

Könnte man das bei Fronius erfragen? Für den WR reicht Leistung zur Steuerung und dann macht ein Merge bei allen Anwendern ohne Ströme im Zähler Fehler...

Habe Zugriff auf Anlagen mit den relevanten drei Zählertypen.
Auch mit Snapinverter und GEN24 Wechselrichtern.

@mdkeil
Copy link
Contributor Author

mdkeil commented Sep 19, 2024

@benesolar
es würde vermutlich schon ausreichen die entsprechende API-Abfrage bei den Anlagen zu machen.. Die Frage ist auch, ob der "Gridmeter" immer auf DeviceID 0 registriert ist.

http://{{ .host }}/solar_api/v1/GetMeterRealtimeData.cgi?Scope=Device&DeviceId=0&DataCollection=MeterRealtimeData

@benesolar
Copy link
Contributor

Zählertypen:
Smart Meter TS 100A-1 = 1phasig, Direktmessung
Smart Meter TS 63A-3 = 3phasig, Direktmessung
Smart Meter 50kA-3 = 3phasig, Wandlermessung

Wechselrichter:
GEN24 (Symo GEN24, Tauro und Verto = alle mit dem Pilot Datamanager)
Snapinverter (Symo, Symo Advanced, Eco = alle mit Datamanager 2.0)
(Symo Hybrid hab ich keinen mehr)

"Meter_Location_Current": 1, = Zählung von einem Fremdwechselrichter
"Meter_Location_Current" : 0.0, oder 0 = Zähler an der Wurzel

Device ID:
Wenn nur ein Zähler am Wechselrichter angeschlossen ist sollte das immer 0 sein.
Möglich sind max. 4 Zähler am Bus.

GEN24 Solar API deaktiviert.txt
GEN24 ohne Zähler.txt
DM2.0 ohne Zähler.txt
DM2.0 mit 1phasigem TS Zähler für Fremdwechselrichter.txt
GEN24 mit 3phasigen Zähler Wandlermessung Wurzel.txt
GEN24 mit 3phasigen Zähler TS Wurzel.txt

@mdkeil
Copy link
Contributor Author

mdkeil commented Sep 19, 2024

GEN24 Solar API deaktiviert.txt

erwartetes Verhalten - aber auch unabhängig vom usage: // ohne aktivierte API kein Auslesen möglich!

GEN24 ohne Zähler.txt

erwartetes Verhalten; auch unabhängig vom WR - hier würde man aber auch kein usage: grid nutzen

DM2.0 ohne Zähler.txt

erwartetes Verhalten; auch unabhängig vom WR - hier würde man aber auch kein usage: grid nutzen

DM2.0 mit 1phasigem TS Zähler für Fremdwechselrichter.txt

Für L2 und L3 müsste eine entsprechende Abfrage rein, die dann L2 und L3 auf 0 setzt, da die Datenpunkte nicht vorhanden sind

GEN24 mit 3phasigen Zähler Wandlermessung Wurzel.txt

erwartete Ausgabe, unabhängig vom WR

GEN24 mit 3phasigen Zähler TS Wurzel.txt

erwartete Ausgabe, unabhängig vom WR

Meiner Ansicht nach, muss nur der Fall abgefangen werden, wenn keine 3P gemessen werden. Der Rest bedingt zwingend einen Zähler, wenn man usage: grid nutzt. Mir fällt zumindest keine usecase ein, wo man dies ohne Zähler tun sollte. Die Frage wäre noch, wenn jemand einen S0-Zähler verwendet als Gridmeter-- aber ich denke, der wird vermutlich ebenso auszulesen sein.

@benesolar
Copy link
Contributor

S0 Zähler können wir vergessen.

Damit wissen wir die Stromflussrichtung nicht und erhalten bei einem 3phasigem Anschluss nur die Summenleistung.

@mdkeil
Copy link
Contributor Author

mdkeil commented Sep 20, 2024

Da ich nicht genau bewerten kann, was schlussendlich besser ist, habe ich den test auf null eingefügt, dass bei fehlendem Datenpunkt der entsprechende Strom auf 0 (wäre null ggfs besser?) gesetzt wird.. aus dem Bauch heraus würde ich sagen, dass der Test auf != null vermutlich besser wäre und dann einfach kein Wert gesetzt wird. Das kann aber @andig vermutlich besser beurteilen.

@andig
Copy link
Member

andig commented Sep 22, 2024

In Anbetracht von #16260 würde ich dazu tendieren, lieber Modbus statt SolarApi zu empfehlen und daher an dieser Stelle nichts zu ändern.

@mdkeil
Copy link
Contributor Author

mdkeil commented Sep 22, 2024

habe ich mir bald gedacht.. dann lassen wir es so.

@andig andig closed this Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants