-
-
Notifications
You must be signed in to change notification settings - Fork 759
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
cli: add battery mode option #10901
cli: add battery mode option #10901
Conversation
/cc @VolkerK62 bzgl. Doku und Allgemeinwissen 🙂 |
Kann ich zwar mangels geeignetem Testobjekt noch nicht selbst testen, aber ja, sieht plausibel aus 👍🏼 |
Wie genau sollen wir das testen? Ich meine, dass Du in einem anderen issue auch ein Tool zum Auslesen der modbus-Register (für mac/linux?) genannt hattest. Sollten wir das auch verwenden oder ist das mit der cli-Option überflüssig? |
Der CLI Befehl erlaubt einfach gezielt einen Batteriemodus einzustellen, ggf. mit trace log, um zu schauen ob das das gewünscht Ergebnis bringt. Damit muss nicht mehr auf Fahrzeugladung oder irgendwelche anderen Laufzeitevents gewartet werden. |
Kannst Du bitte kurz erklären, was die Optionen für eine Auswirkung haben?
|
Ich hab das jetzt mal ausgeführt, werde aber aus den logs nicht schlau.
|
Zum Sofar Support: Zunächst ist mir aufgefallen, dass das Register für die Entladungstiefe falsch ist. Siehe mein Review Kommentar bei #10847. Nun fürchte ich jedoch, dass auch eine Korrektur nicht viel hilft. Ich habe nun mit Hilfe meiner HomeAssistant Integration versucht die Entladungstiefe zu ändern und hier wird auch das korrekte Register benutzt (siehe Source Code der Home Assistant integration: https://github.com/wills106/homeassistant-solax-modbus/blob/5730eb7e1228aae86a3886abcbf5ad5c18f3163a/custom_components/solax_modbus/plugin_sofar.py#L238). Aber leider habe ich auch damit keinen Erfolg. Ob im Self Use Mode oder im Passive Mode - ich kann da ändern wie ich es will - im Wechselrichter ändert sich der Wert einfach nicht. Irgendwas stimmt da nicht oder ich müsste noch irgendwo einen anderen Modus setzen. Solange ich es aber nicht hinbekomme, den Wert direkt über HomeAssistant zu setzen, mach der Support in EVCC keinen Sinn. Ich empfehle daher #10847 zu reverten oder mindestens das Register zu korrigieren. |
nochmal Anfrage an den Support probieren? |
Ist behoben |
@andig - es kommt langsam Licht in die Sache, dank Photovoltaikforum. Auch beim setzen des DoD Levels scheint Sofar mal wieder Sonderwege zu gehen, wie auch schon beim setzen anderer Werte: ![]() D.h. für einen Sofar Support brauchen wir hier einen anderen, komplizierteren Ansatz, der eher in Richtung Scripting gehen würde. Mein Vorschlag: Nehmt den Sofar Support für Battery Control raus. Wer Battery Control mit SofarSolar realisieren will muss dass eben mit Home Assistant machen. |
Das mit den 19 auf einmal würde ich erstmal bedingt glauben. Den zweiten Hinweis- auch BatConfig_Control zu schreiben kannst Du ja ausprobieren. Ich würde erst wieder etwas ändern wenns eine belastbare Grundlage gibt (ausser dass die aktuelle Funktion wieder ausgebaut wird: #11341). |
@cschlipf hast Du das mal ausprobiert? Sonst würde ich Sofar jetzt wieder rauswerfen... |
@andig - gerade mal mit der Home Assistant Integration getestet. Tut nicht. Vielleicht kennt ja jemand anders einen Trick. |
@andig Ich habe jetzt die Doku und ich kann Euch definitiv sagen, dass ihr für Sofar das nicht umsetzen wollt. Das ist die Liste der Parameter, die man schreiben muss, nur um den Depth of Discharge zu schreiben: ![]() Es sind nicht 19 Register, sondern 23. Ich sag mal: Das wollt ihr nicht ;-) |
Kannst du mir das gesamte Dokument mal bitte anhängen oder per Slack schicken? |
Bitte sehr. |
Useful for testing batter control:
Usage: