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

Fix pv charge regulation #1406

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

vuffiraa72
Copy link
Contributor

Passend zum Issue #1405. Es sind 3 einzelne Commits. Die können dann auch einzeln per Cherry-Pick übernommen werden. Da ich nicht ganz sicher bin, ob das in allen Konstellationen passt. Ich teste bei mir nur mit PV und einer openWB, ohne Speicher und komplexer Lastenverteilung.

Zu den Commits:

  • test calculation of current:
    Die ersten beiden Testkonfigurationen sind aus meinem Log aus dem Beitrag https://openwb.de/forum/viewtopic.php?p=99654#p99654 entnommen:

    • 2024-01-08 14:29:00,510: 84,08W Einspeisung -> EVSE bereinigt soll die Ladung auf 8,57A erhöht werden -> entspricht der kompletten verfügbaren Solarleistung bei 230V Spannung
    • 2024-01-08 14:29:11,463: 41,96W Bezug -> EVSE bereinigt soll die Ladung wieder auf 8,01A vermindert werden -> entspricht der verfügbaren Solarleistung - Regelmittelpunkt von 115W bei 230V Spannung

    Die weiteren Testkonfigurationen sind dann idealisierte Zustände mit 2300W verfügbarer Solarleistung. Es wird getestet, dass die berechnete Ladeleistung innerhalb den Regelbereichs liegt, die Grenzwerte sind nicht zulässig. Das soll verdeutlichen, dass die Regellogik hier nicht ganz ausgeglichen ist.

  • respect actual voltages while calulating surplus
    Berechnung mit der wirklich anliegenden Spannung

  • rage offset is independent from state of control range
    Regelmodus wird immer beachtet

@LKuemmel LKuemmel added this to the 2.1.4 milestone Feb 20, 2024
@LKuemmel LKuemmel removed this from the 2.1.4 Step 1 milestone Mar 22, 2024
@SREbuilt
Copy link

wäre schön, wenn diese Änderungen es ins Release schaffen.
Habe ähnliches festgestellt: Sägezahnladen.

@fibroin
Copy link

fibroin commented May 7, 2024

Hallo,
auch ich fände es gut, wenn diese Anpassung noch einmal Beachtung finden würde.
Es scheint, dass der bisherige Algorithmus noch den Regelbereich beim PV-Laden nicht wirklich berücksichtigt.

Gibt es irgendwelche Probleme bei dem Fix?

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.

4 participants