-
-
Notifications
You must be signed in to change notification settings - Fork 833
Loadpoint: allow fine-grained battery boost #19385
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
base: master
Are you sure you want to change the base?
Conversation
Da ich es momentan mangels Zeit nicht genauer analysieren kann hier meine Gedanken dazu: Es braucht gewisse Toleranzen, da hier verschiedene Dinge gleichzeitig einwirken.
In meinem Test (Wallbe mit 0,1A Schritten, ZOE, SMA SBS) funktioniert dieser PR mit diesem Stand nicht. Vielleicht haben @mfuchs1984 und @VolkerK62 auch noch eine Idee dazu. Irgendwo hatten wir das schon mal analysiert und diskutiert woraus dann die aktuelle (funktionierende) Lösung entstand. Da ging es speziell auch um die Abhängigkeit mit der Phasenanzahl. Leider finde ich das nicht mehr. |
Das Hochfahren passiert ja schon vorher. Hier gehts nur ums nachsteuern.
Dann müssten wir ja jetzt einen plausiblem Totbereich raus finden. Schaffen wir das? Zumindest bei 3p sind 690W schon sehr heftig. Wäre großartig wenn das jemand mit Akku testen könnte- ich kanns mangels Gerät leider nicht. |
Der Totbereich spielst doch im normalen Betrieb auch eine Rolle? |
Klingt plausibel. ...dann sollten die 100W aus diesem PR ja reichen. Egal wie man es dreht: es muss jemand schauen was da gerechnet wird und warum es nicht funktioniert. |
Hier ist ein Fall, wo trotz boost, die Ladeleistung kontinuierlich sinkt
Müsste man zum delta nicht auch noch Line 1317 in 69bfa6d
|
Klingt logisch. Bei mir steht Vielleicht letztendlich mit |
Night schön, aber probehalber umgesetzt |
Das wirkt dann aber nur (passend zum PR) bei mA-Regelung. |
Bei ganzen A sind es soweiso mindestens 230W. Da damit bisher kein Problem bekannt ist auch keine Änderung- oder worauf willst du hinaus?
Was schlägst Du vor? |
siehe oben #19385 (comment) |
Verstehe ich nicht. Im normalen fall (nicht mA) werden zum "richtigen" Leistungsbedarf ja schon 1A drauf gerechnet. Wieso sollte man da residual Power nochmal rein rechnen? Die ist ja schon drin! |
Im Zweifel: gerne PR direkt anpassen und ausprobieren. |
Mit meine Idee wäre die Änderung bei |
Ich verstehe nur bahnhof. |
Wenn es mal wieder Sonne gibt und in meiner Batterie etwas drin ist, dann werde ich es mal ausprobieren. |
da ich nur einen 1A-Charger habe, passt es nciht in diesen PR. |
Ich habe einen neuen Commit zur kritischen Diskussion gemacht. |
Co-authored-by: premultiply <[email protected]>
Gabs schon eine Chance, das zu testen? |
Friendly ping |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enables fine-grained battery boost by considering mA charging and adjusting the boost power calculation accordingly. In summary, the PR updates the Prepare API for loadpoints to pass a site parameter, adjusts boost power computation in the Loadpoint, and revises tests to create a Site instance.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
core/site.go | Updated the Prepare invocation to pass the Site instance. |
core/loadpoint_test.go | Modified test calls to use a new Site instance for the Prepare call. |
core/loadpoint.go | Added a site field, modified Prepare signature, and updated boostPower calculation to incorporate residual power. |
Comments suppressed due to low confidence (1)
core/loadpoint_test.go:85
- [nitpick] Ensure that using new(Site) in tests provides a fully functional or properly mocked site.API implementation. Inadequate initialization may affect the validity of the test outcomes.
lp.Prepare(new(Site), uiChan, pushChan, lpChan)
// push demand to drain battery | ||
delta := lp.EffectiveStepPower() | ||
// push demand to drain battery (at least 100W) | ||
delta := math.Max(100, math.Abs(lp.site.GetResidualPower())) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Macht das Abs hier Sinn? Warum sollte man eine negative residualpower hier überhaupt berücksichtigen und in eine positive verwandeln?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Konfigurierte negative residualPower
bedeutet ja dass man immer (leichten) Netzbezug statt Einspeisung am Regelpunkt haben will. Der Arbeitspunkt (dH der neue Nullpunkt) ist verschoben.
Auch wenn negative residualPower eher ein Spezialfall ist muss man dabei um aus einem stationären Zustand zu kommen imho auch den Arbeitspunkt noch weiter in Richtung Netzbezug verschieben.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das verstehe ich. Aber warum sollte man dann hier positive residual power genauso behandeln? Weil die kompensiert werden muss da es sonst zu keinem Bezug kommt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja genau. Durch positive residualPower
(der Standardfall) liegt der Arbeitspunkt ja dementsprechend im "Einspeisebereich".
Das wird bzw. muss im Boost-Betrieb in den "Bezugsbereich" gespiegelt werden, damit der Batteriespeicher seine Entladeleistung erhöht.
(Diese wiederum muss so groß sein dass ein LP in die nächste A-Stufe schalten kann könnte - wenn der Speicher noch nicht an seinem Limit angekommen ist.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dann fehlt nur noch Test ;)
friendly ping |
Fix #19845
Currently, battery boost draw at least 1A per phase from grid, regardless if mA charging is available. This PR honors mA charging.