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

Huawei SUN2000: handle cascading inverters #17726

Merged
merged 9 commits into from
Dec 14, 2024
Merged

Conversation

Hypo93
Copy link
Contributor

@Hypo93 Hypo93 commented Dec 13, 2024

Beschreibung des Fixes
Beim gestrigen Update des Huawei-SUN2000-Templates habe ich nicht berücksichtigt, dass in bestimmten Wechselrichter-Konstellationen (z.B. Wechselrichter-Kaskaden, bei denen andere AC-angebundenen Wechselrichter den Batteriewechselrichter speisen) das AC-Laden dauerhaft aktiv bleiben muss.

Um dieses Verhalten anzupassen, habe ich eine neue Konfigurationsmöglichkeit über die Variable cascade eingeführt:

cascade: "true" → AC-Laden bleibt aktiv.
cascade: "false" → Standardverhalten.

Diese Einstellung kann nun über die evcc.yaml-Datei oder direkt im Konfigurations-UI angepasst werden.

Links
Verwandtes Issue: #17692
Pull Request: #17695

Änderungen
Neue Variable: cascade für die Konfiguration des AC-Ladeverhaltens.
Anpassung des Huawei-SUN2000-Templates, um die neue Option zu unterstützen.

Testanleitung
Konfiguriere die Variable cascade: "true" in der evcc.yaml oder über das UI.
Überprüfe das Verhalten in Wechselrichter-Kaskaden, bei denen der Batteriewechselrichter über AC versorgt wird.

Hintergrund
Mit dieser Anpassung wird sichergestellt, dass das AC-Laden in komplexeren Wechselrichter-Setups korrekt funktioniert.

Add configuration objekt "cascade"
Add description for Huawei-SUN2000 configuration object "cascade"
Hypo93 added a commit to Hypo93/docs that referenced this pull request Dec 13, 2024
Hinzufügen der Variable "cascade" für die Dokumentation des PR evcc-io/evcc#17726
Hypo93 added a commit to Hypo93/docs that referenced this pull request Dec 13, 2024
Hinzufügen der Variable "cascade" für die Dokumentation des PR evcc-io/evcc#17726
@andig
Copy link
Member

andig commented Dec 14, 2024

Der Parameter braucht bitte eine Erläuterung. Ich bin auch mit dem Namen nicht glücklich, es sei denn wir sind uns 100%ig sicher dass dies der einzige Anwendungsfall ist.

@andig andig marked this pull request as draft December 14, 2024 10:23
@andig andig added the devices Specific device support label Dec 14, 2024
@Hypo93
Copy link
Contributor Author

Hypo93 commented Dec 14, 2024

Der Parameter braucht bitte eine Erläuterung. Ich bin auch mit dem Namen nicht glücklich, es sei denn wir sind uns 100%ig sicher dass dies der einzige Anwendungsfall ist.

Moin Andi, ich hab den Parameter einen modellspezifischen eindeutigen Namen gegeben, damit wir sicher sind, dass er sonst nirgends anders genutzt wird. Wo möchtest du noch eine Erläuterung haben für den Parameter? Fürs docs repository hab ich es schon vorbereitet aber noch kein PR gestartet

util/templates/defaults.yaml Outdated Show resolved Hide resolved
@andig andig marked this pull request as ready for review December 14, 2024 11:34
@andig
Copy link
Member

andig commented Dec 14, 2024

LGTM. Jetzt bleibt noch die Frage nach dem Default. Das alte Verhalten vor dem letzten PR als Default? Oder (hier) das neue?

@andig andig changed the title Anpassung für Wechselrichterkaskaden AC-Laden - Huawei SUN2000 + LUNA Huawei SUN2000: handle cascading inverters Dec 14, 2024
@Hypo93
Copy link
Contributor Author

Hypo93 commented Dec 14, 2024

Ich hatte mir gedacht, das neue als default, da die Werkseinstellung von Huawei auch AC-Laden disabled ist. Auf enable muss der WR nur gestellt werden, in der Sonderkonstellation mit mehreren WR als Kaskade, wenn es gewollt ist über AC angebundenen WR in den Akku zu speisen (also aus meiner Sicht nicht der Standard).

util/templates/defaults.yaml Outdated Show resolved Hide resolved
@andig andig merged commit b14faf0 into evcc-io:master Dec 14, 2024
6 checks passed
@Hypo93 Hypo93 deleted the huawei-fix branch December 14, 2024 12:34
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.

2 participants