Skip to content
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

Restore logic for chargers that disable themselves to be able to switch phases #12847

Merged
merged 3 commits into from
Mar 11, 2024
Merged

Restore logic for chargers that disable themselves to be able to switch phases #12847

merged 3 commits into from
Mar 11, 2024

Conversation

MarkusGH
Copy link
Contributor

@MarkusGH MarkusGH commented Mar 10, 2024

Fix #12825

Restore logic for chargers that disable themselves to be able to switch phases
Fix formatting
@MarkusGH MarkusGH marked this pull request as ready for review March 10, 2024 13:24
@andig andig requested a review from GrimmiMeloni March 10, 2024 13:42
@andig andig added the bug Something isn't working label Mar 10, 2024
@andig
Copy link
Member

andig commented Mar 10, 2024

Fix for: #12825

@MarkusGH einfach "fix #12825", sonst wird das nicht sauber verlinkt.

@MarkusGH
Copy link
Contributor Author

Wobei man syncCharger einfach mal gründlich ausmisten sollte.
Das ist viel Code für eine mittlerweile im Grunde einfache Logik.
Soll ich einen Vorschlag machen, wenn der PR durch ist?

@andig
Copy link
Member

andig commented Mar 10, 2024

Sehr gerne- ich bekomme da auch immer graue Haare. Hier hätte ich gerne noch eine Meinung von @GrimmiMeloni

@MarkusGH
Copy link
Contributor Author

MarkusGH commented Mar 10, 2024

@GrimmiMeloni: Ist es für die Easee OK, wenn sie unter Umständen gleich nach dem "Pausieren" wieder "Resumed" wird? Denn das passiert jetzt im nächsten Regelintervall, gleich nach der Phasenumschaltung. "Früher" war da eine Mindestverzögerung von 30 Sekunden drin, die ich als "Angstlösung" eingebaut hatte.

@GrimmiMeloni
Copy link
Collaborator

Die Phasenumschaltung bei einzelner Easee benötigt kein Pause/Resume. Wir setzen einfach die Currents auf dem Circuit, und die Box kümmert sich selbst um alles. Bei mehreren Easee Boxen machen wir es ähnlich, da braucht es aber anscheinend einen Resume. In jedem Fall sollte die Easee aber keinen Schmerz damit haben direkt wieder aktiviert zu werden.

Bzgl. syncCharger ausmisten wäre ich eher zurückhaltender. Wir haben da einige Iterationen durch, und ich bin skeptisch das man da ohne Nebenwirkungen vereinfachen kann. Versuchen können wir es aber sicherlich.

@andig
Copy link
Member

andig commented Mar 10, 2024

@GrimmiMeloni Änderung für Dich ok?

@MarkusGH
Copy link
Contributor Author

Bzgl. syncCharger ausmisten wäre ich eher zurückhaltender. Wir haben da einige Iterationen durch, und ich bin skeptisch das man da ohne Nebenwirkungen vereinfachen kann. Versuchen können wir es aber sicherlich.

Die berücksichtigten Fälle sind nicht weiter reduzierbar - aber die Logik ist mittlerweile völlig unübersichtlich geworden.

@andig andig merged commit f8a10ae into evcc-io:master Mar 11, 2024
6 checks passed
@MarkusGH
Copy link
Contributor Author

@andig, @GrimmiMeloni: Here you go: #12876

@MarkusGH MarkusGH deleted the patch-2 branch March 11, 2024 18:40
thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

easee 3p -> 1p Phasenumschaltung schaltet nicht auf 1p runter, sondern oszilliert zwischen 3p und nix
3 participants