-
-
Notifications
You must be signed in to change notification settings - Fork 802
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
Loadpoint: allow fine-grained battery boost #19385
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. |
@@ -1330,7 +1337,7 @@ func (lp *Loadpoint) boostPower(batteryBoostPower float64) float64 { | |||
} | |||
} | |||
|
|||
res := batteryBoostPower + delta | |||
res := batteryBoostPower + delta + 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.
Anm: Hier wird (ggf. nochmals) residualPower
addiert um dessen Offset an der site
zu neutralisieren (sonst verschobener Arbeitspunkt, #19845).
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.
Ich kann dir nicht folgen. Ist das so jetzt richtig? Dann müsste es noch getestet werden :O
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, sollte nun so richtig sein.
Gestern war leider für mich keine Testmöglichkeit in der Praxis mehr.
Ich habe einen neuen Commit zur kritischen Diskussion gemacht. |
Co-authored-by: premultiply <4681172+premultiply@users.noreply.github.com>
Gabs schon eine Chance, das zu testen? |
Fix #19845
Currently, battery boost draw at least 1A per phase from grid, regardless if mA charging is available. This PR honors mA charging.