-
-
Notifications
You must be signed in to change notification settings - Fork 555
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
Wattpilot: refactored locking to avoid deadlock #14498
Conversation
mabunixda
commented
Jun 21, 2024
•
edited
Loading
edited
- updated wattpilot to 1.8.0 to handle wattpilot reconnect issues and wattpilot crashes on multiple parallel writes on websocket
Signed-off-by: Martin Buchleitner <mbuchleitner@infralovers.com>
Signed-off-by: Martin Buchleitner <mbuchleitner@infralovers.com>
Signed-off-by: Martin Buchleitner <mbuchleitner@infralovers.com>
* master: (34 commits) Warp3: require automatic 1p phase switching to be disabled (#14494) Mercedes: add missing region to template (#14465) SofarSolar: fix batterys power sign (#14448) Tariffs: fix charges not applied (#14399) docs: circuit link chore: improve e2e stability chore: use vehicle-common include (#14452) Improve token generation config output (#14454) Revert "Solis-Hybrid: fix battery power (#14423)" (#14453) Solis-Hybrid: fix battery power (#14423) chore: improve log legibility Circuit: apply current rounding to circuit limits, too Circuits: better logging Revert "Renault: update api" Circuit: format powers with 5 digits chore: minor Site: remove root circuit configuration (#14244) Circuits: consistent log formatting Em2go: add api.CurrentGetter Always make fatal errors json-serializable (#14396) ...
Signed-off-by: Martin Buchleitner <mbuchleitner@infralovers.com>
Signed-off-by: Martin Buchleitner <mbuchleitner@infralovers.com>
@mabunixda without looking at the code: that should not be done for evcc or it would stop the transaction right away? |
Die Transaktion - also das Laden - wird nun unmittelbar gestartet wenn ein bekanntes Auto angesteckt wird. Das Verhalten ist ja so auch bei anderen Charger-Implementierungen. Soll ich das in die Konfiguration legen, dass man das Verhalten aktivieren kann? |
Reden wir hier aneinander vorbei? Das ist bei keiner Implementierung der Fall. Woher soll der Charger wissen, ob z.B. PV Modus eingestellt und genug Leistung vorhanden? Das ist allein Aufgabe vom Ladepunkt! |
Signed-off-by: Martin Buchleitner <mbuchleitner@infralovers.com>
Ja stimmt - das TRX Flag ist aber um die Authentifizierung zu bestätigen. Ansonsten funktioniert das Enable nicht, da bei Eingeschaltener Authenfizierung ansonsten das Laden nicht startet sie wie es implementiert war. Ich splitte diese 2 Dinge nun aber mal in 2 PRs - dieser ist nun rein für das Reconnect/Deadlocking |
M.E. sollte Authentifizierung deaktiviert werden (das kann gerne in die Doku)- die lässt sich ja über evcc mittels Identifier auch realisieren? |