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

SonnenBatterie: add support for time-of-use operating mode #18131

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

rivengh
Copy link
Contributor

@rivengh rivengh commented Jan 8, 2025

Currently evcc only supports the self-consumption mode of the sonnenBatterie. This PR adds support for the time-of-use mode for users who operate their battery in this mode.

By default, evcc still assumes self-consumption, but the mode can be selected with the operatingmode parameter.

@andig
Copy link
Member

andig commented Jan 8, 2025

In other templates we've called this defaultmode although I like operatingmode better- does it make sense to align these (...using defaultmode)?

@andig andig added the devices Specific device support label Jan 8, 2025
@rivengh
Copy link
Contributor Author

rivengh commented Jan 9, 2025

Thanks, I hadn't seen that. The parameter name has been changed to defaultmode. Can we keep the choice type here or should it also be converted to integer as in the other templates?

@andig
Copy link
Member

andig commented Jan 9, 2025

Choice seems the better (choice). Thank you!

@andig andig merged commit 9308876 into evcc-io:master Jan 9, 2025
6 checks passed
@marleyan333
Copy link

Currently evcc only supports the self-consumption mode of the sonnenBatterie. This PR adds support for the time-of-use mode for users who operate their battery in this mode.

By default, evcc still assumes self-consumption, but the mode can be selected with the operatingmode parameter.

Würde das nicht auch mein Problem mit dem Netzladen beheben? Ab wann wird denn diese Änderung umgesetzt? Ist das jetzt im nightly Build?

@rivengh rivengh deleted the sb-support-time-of-use branch January 10, 2025 14:31
@rivengh
Copy link
Contributor Author

rivengh commented Jan 10, 2025

Würde das nicht auch mein Problem mit dem Netzladen beheben?

Nein, das Problem mit der Netzladung (#16550) wird dadurch nicht gelöst. Mit diesem PR ist es jedoch möglich, gleichzeitig (a) den Time-of-Use Modus der Sonnenbatterie zur Steuerung der Netzladung durch die Batterie selbst und (b) die Entladesperre von evcc zu nutzen (Option "Verhindere Entladung im Schnell-Modus und bei geplantem Laden").

Hintergrund: Die Netzladung wird durch den täglichen Reset der Sonnenbatterie unterbrochen. Nach meiner derzeitigen Analyse bleibt der Manual Modus der Sonnenbatterie dabei jedoch erhalten, so dass nur die experimentelle Netzladefunktion betroffen ist, nicht aber die Entladesperre selbst. Dies entspricht auch dem Verhalten aus dem ursprünglichen Issue #16550.

@marleyan333
Copy link

marleyan333 commented Jan 10, 2025 via email

@rivengh
Copy link
Contributor Author

rivengh commented Jan 10, 2025

Ich verstehe das Feature der Entladesperre ehrlich gesagt nicht ganz [...]

Eine Erklärung der Funktion findest du hier in der Dokumentation:
https://docs.evcc.io/docs/features/battery#aktive-batteriesteuerung
Oder auch in diesem Blogeintrag:
https://docs.evcc.io/blog/2024/02/01/v0.124-new-tesla-api#entladesperre-beim-schnellladen

Und was heißt eigentlich PR?

PR ist die Abkürzung für Pull Request. Siehe dazu: https://docs.github.com/de/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests#about-pull-requests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants