-
-
Notifications
You must be signed in to change notification settings - Fork 546
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
Modbus plugin: split modbus and sunspec (BC) #11663
Conversation
b2b0bd0
to
f328e0b
Compare
@premultiply wenn Du Lust hast könntest Du mal versuchen, ob Modbus LSW first Writes funktionieren? |
@premultiply sunspec writes sind auch drin. |
Ich hab leider auch keinerlei Gerät sowas braucht. |
3b1b2f2
to
9a93c10
Compare
W (AC-Power) wäre bei Hybrid immer falsch. |
Ich kenne mich mit der Schreibweise nicht aus, aber wenn die Umrechnung aufs gleiche Register führt, dann wird das wohl passen.
Ja. Hab nur abgekürzt bei mir. ;) Ich würde die Remote-Steuerung einfach dauerhaft aktivieren und alles darüber machen. Also evcc setzt beim Start (oder alternativ per cli) die Register um die Remote-Control zu aktivieren:
Deaktivieren könnte man das ganze dann per cli, falls man evcc nicht mehr benutzen möchte. Ansonsten läuft aber auch einfach der timeout ab und dann sollte der WR sowieso in evcc könnte beim graceful beenden natürlich auch evcc steuert dann den Akku, indem es regelmäßig den
Wie man das Maximum fürs Charge/Discharge Limit herausfindet, bin ich gerade nicht sicher. Vlt. einstellbar machen? |
Das wollen wir nicht. Keine merkwürdigen Konstanten. Entweder geht das über den Timeout oder es geht halt nicht. |
@MarcusWichelmann StorageConf_CtrlMode: 0xe004 = 4 (Remote) ist jetzt drin, beim Rest würde ich auf sinnvolle Defaults vertrauen. Ansonsten gerne mal testen- das wäre es jetzt von meiner Seite. Wenn das mangels magischer Konstanten nicht funktioniert würde ich SE hier wieder raus nehmen bis jemand eine bessere Idee hat. |
Okay verstehe. Ja dann ist vermutlich so besser: Initial:
Je nach Modus:
Das Das hat auch den Vorteil, dass der Wechselrichter während des "normal" Modes von evcc in einem Zustand ist, als würde man garkein evcc benutzen. evcc greift nur ein beim "hold". |
Entschuldigung, dass ich mich hier auch noch einmische, aber ich halte es nicht für gut (und auch nicht für notwendig) StorageConf_CtrlMode automatisch ständig neu zu schreiben. Denn das ist eine Konfigurationsoption, die auch in der SetApp sichtbar ist. Aus meiner Sicht ein guter Anfang wäre: Beim Start oder per CLI:
Dann laufend je nach Modus:
Denn dann kann der Anwender wenn was schief geht selber wieder per SetApp StorageRemoteCtrl_CommandMode = 1 setzen. Maximum kann aus meiner Sicht alles >= 5000 sein, die Wechselrichter können eh' nicht mehr. Bei mir stand das per Default übrigens auf 16384. |
Bei neuerer Software scheint StorageConf_CtrlMode=4 per UI übrigens gar nicht einstellbar zu sein, da gibt es wohl einen Mode 5 (Remote + TimeOfUse), der für unsere Zwecke hier auch funktionieren würde. |
Aussage oben: Anwender kann es nicht setzen. Irgendwo muss es also her kommen.
Davon ist auch nicht die Rede- siehe Template. Long story short: ich nehme SE hier wieder raus und wir können das gerne separat weiter diskutieren, es scheint nicht als würde es hier eine Einigung geben. Weiter mit SE in #11904. |
Verwirrung im Endstatium 😄 Ich mache, wenn ich wieder da bin (2-3h) nen neuen Merge Request, dann können wirs da diskutieren. |
Spitze - und sorry für das reingrätschen! |
Ist das der gleiche Wert, der in einem dieser Batterie-Register steht? |
Ich freue mich, wenn der Scale wieder funktioniert. Den benötige ich, um immer einen minimalen Überschuss ins Netz zu haben. `
|
Purpose of this PR is to increase flexibility for the
modbus
plugin to allow more write operations. Replaces #11166, #11633TODO
fallback
plugin (Add fallback plugin #11773)modbus
LE writessunspec
writesOut of scope
modbus
meter- this is actually acustom
meter withmodbus
plugin (replace bymbmd
meter for remaining MBMD devices) chore: rename modbus meter to mbmd #11665