-
-
Notifications
You must be signed in to change notification settings - Fork 829
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
Add active battery control deye-hybrid-3p.yaml #12415
Conversation
source: sequence | ||
set: | ||
- source: const | ||
value: 100 # SoC TOU 1 100% |
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.
Dito
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, deswegen auch draft
.
Wie charge umzusetzten wäre, müsste auch noch eruiert werden. Es scheint mir, als müssen da wieder mal Leistungswerte gesetzt werden, was ATM nicht gemacht wird.
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.
Kannst du mir einen Tipp geben, wie ich auf den Parameter soc
zugreifen kann und diesen dann auf Register 166
setzen kann?
Hier wird es ja so gemacht, aber ich werde da nicht schlau daraus.
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, deswegen auch
draft
.Wie charge umzusetzten wäre, müsste auch noch eruiert werden. Es scheint mir, als müssen da wieder mal Leistungswerte gesetzt werden, was ATM nicht gemacht wird.
Laden der Batterie nach Ziel soc würde über Register 172
funktionieren. Das wäre auch aus dem time of use Zeitslot 1 und muss dazu auf 1
gesetzt werden.
Allerdings muss hierzu zuerst in den Einstellungen das Laden der Batterie aus dem Netz global aktiviert werden. Das wäre Register 130
.
Damit sich dieses aber aktivieren lässt, muss zuerst der Strom für das Netzladen und ein soc konfiguriert werden. (Register 127
und 128
) werden. Siehe auch hier in der Anleitung:
Mein Vorschlag wäre, dass evcc nur Register 172
setzt und die anderen beiden Register von Benutzer manuell gesetzt werden müssen.
Eine andere Möglichkeit hätte ich jetzt für den Deye nicht rausgefunden.
Sorry, ich komme hier nicht wirklich weiter. So wie es aktuell ist, würde es funktionieren, wenn Das ist aber anscheinend nicht vorgesehen, bzw es wird nur der Teil mit Ich verstehe halt den grundlegenden Mechanismus noch nicht, wie die Templates genau funktionieren. |
Es gibt 2 unterschiedliche Ansätze- entweder über limitsoc implizit oder über batterymode. Die sind nciht zu kombinieren. |
Ja, das habe ich auch festgestellt. Daher wäre es das beste, über Ich verstehe hier nur nicht, wie ich auf de Variable des socs zugreifen kann um die Batterie in Einfach auf 100% setzen würde zwar auch passen, aber halt nicht schön.... Kannst du mir da einen Tipp geben? |
Das geht aktuell nicht :/ |
Dann muss es so sein, wie es zu Beginn war. Also bei Bei Es wäre jetzt nicht jedes mal notwendig die tou Zeiten neu zu schreiben. Warum der UI check fehlschlägt kann ich mir nicht erklären. Ist mein erster ernsthafter PR, ich hoffe daher, dass ich nicht allzuviel falsch gemacht habe.... |
Den letzten musst Du bitte selbst ändern:
Du hast den Zeilenumbruch geklaut... |
Hallo, muss den PR nochmal öffnen: Das Template funktioniert in meinem Setup (Deye Sun-12K-SG04LP3) zwar prinzipiell, hat aber sehr unschöne bis kritische Nebeneffekte:
Gerade der letzte Punkt bereitet mir Bauchschmerzen, ein Ziel SoC von 0 (oder irgendwas undefiniertes) ist in keinem Fall wünschenswert. Vielleicht ist das auch das was in #12963 festgestellt wurde. Prinzipiell sollte evcc mMn bei Rückkehr zu Ich hätte noch einen alternativen Vorschlag: Für |
Blöd aber akzeptabel wenn wir das im Template dokumentieren. Wer ToU braucht kann das dann eben nicht verwenden.
In anderen Templates die über
Das geht mit Templates nicht... |
Das scheint mir, wie @andig schon geschrieben hat, eine Eigenheit des WR zu sein. Geschrieben wird nur TOU1. Warum er dann die anderen auch mit ändert. liegt nicht in dem Einflussbereich von evcc.
Wann genau passiert das? Wenn evcc von Der
Nach meinen Kenntnissen ist es so, dass das Kernteam mal vor einiger Zeit beschlossen hat, dass evcc aus Sicherheitsgründen keine Leistungswerte schreibt und dies dem WR überlassen werden soll. Daher die etwas umständliche Lösung über die TOU. |
/cc @premultiply |
Ja, das ist wohl so, wenn du Endzeit von TOU1 setzt, ist das ja gleichzeitig die Anfangszeit von TOU2. Und wenn die dann zeitlich nach der bisher definierten Endzeit von TOU2 (= Anfangszeit TOU3) liegt, setzt der WR TOU3 wohl automatisch auf 0:00.
Ja, genau dann, gerade wieder so passiert:
Habe ich, hier Auszug aus meiner evcc.yaml.
Ok, das ist verständlich, wenn da beim Schreiben ein Fehler passiert, wäre das schon sehr "suboptimal". Noch eine andere Idee: |
Ich habe meine Idee mal getestet und das template entsprechend erweitert, setze also alle SoC-Werte bzw. grid charge Bits der TOU Tabelle bzgl. der Erfordernisse des aktuellen Modus, lasse aber alle Zeitänderungen weg. Erfolgreich getestet in allen Modi Da damit das ursprüngliche Konzept erhalten bleibt, keine ungewünschten "0" Einträge vom WR erzeugt werden und auch die Sicherheitsvorgaben eingehalten werden (kein Schreiben von Leistungen/Strömen), würde ich vorschlagen, dazu einen neuen PR zu starten. @deadrabbit87 , @andig einverstanden? |
Gerne 👍🏻 |
Ich hab kein Problem damit. @premultiply @andig Was aber noch zu klären wäre, ob jetzt definitiv keine Leistungswerte geschrieben werden dürfen oder nicht. Ich hatte hier auch schon mal gefragt, aber keine Antwort erhalten. M.E. würde das bei vielen WR einiges erleichtern. |
Man kann Leistungsgrenzwerte schreiben wenn diese sich zwischen einem definierten Default-Wert und 0 bewegen. Was wir keinesfalls tun werden ist mit evcc ständig irgendwelche Batterieleistungen nach- bzw. ausregeln. Wie immer lassen wir die Batteriesteuerung dann lieber ganz weg bevor es hier Probleme gibt. Das scheint hier möglicherweise auch nötig zu sein? |
Danke - im Fall vom Deye würde ich das trotzdem über die TOU machen. Ich kenn nur Register |
Ja, direkt die Ströme reinschreiben wäre einfacher, aber ich kann die Bedenken nachvollziehen. Nachdem hier keine leistungsbezogenen Größen geschrieben werden, sollte die Verwendung der TOU so passen. Ich schicke gleich den PR raus, @deadrabbit87 , vielleicht kannst du das ja auch mal bei dir testen. |
Da sollte auf jeden Fall nochmal drüber geschaut werden.
Nach ersten Tests funktioniert
hold
undnormal
fix #12333