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

Add my-PV ELWA 2 charger #14504

Merged
merged 9 commits into from
Jun 29, 2024
Merged

Add my-PV ELWA 2 charger #14504

merged 9 commits into from
Jun 29, 2024

Conversation

andig
Copy link
Member

@andig andig commented Jun 21, 2024

Refs #12761

@andig andig added the devices Specific device support label Jun 21, 2024
templates/definition/charger/ac-elwa-2.yaml Outdated Show resolved Hide resolved
templates/definition/charger/ac-elwa-2.yaml Outdated Show resolved Hide resolved
templates/definition/charger/ac-elwa-2.yaml Outdated Show resolved Hide resolved
@premultiply
Copy link
Member

Auch noch die Ist-Leistung (power) mit hinzunehmen als Summe von 1074+1075?
Und enabled vielleicht besser von 1071?

Co-authored-by: premultiply <4681172+premultiply@users.noreply.github.com>
@andig
Copy link
Member Author

andig commented Jun 21, 2024

@premultiply schau mal bitte in die Discussion- ich habs nur nachgebaut.

@TecTim18
Copy link

@premultiply

Auch noch die Ist-Leistung (power) mit hinzunehmen als Summe von 1074+1075?

Wenn die Aux-function (Mode 2) mit genutzt werden können soll, dann macht das Sinn gleich die Summe zu ziehen. Allerdings würde ich die Aux Funktion nicht unterstützen wollen, da dessen Temperatur, Status ect. via evcc nicht sauber gezeigt werden kann, außer man macht sich eine zweiten Loadpoint und füttert diesen, allerdings ohne Einstellmöglichkeiten... Genau, viel zu komplex und von den wenigsten genutzt ;-)

Daher würde ich nur Mode 1 (ohne externesR Relais) unterstützen um eine saubere und übersichtliche Intergration umzusetzen. Damit reicht dann der Wert von 1074 als aktuell aufgenommene Heizleistung.

Und enabled vielleicht besser von 1071?

Wenn 1071 > 0 ist dann ready to heat? Denn der User kann den ELWA je nach Anschlussart auch drosseln. Würde daher kein oberes Ende festlegen.

andig and others added 2 commits June 21, 2024 21:02
Co-authored-by: premultiply <4681172+premultiply@users.noreply.github.com>
Co-authored-by: premultiply <4681172+premultiply@users.noreply.github.com>
@andig
Copy link
Member Author

andig commented Jun 21, 2024

Enable really sets the power to 1W now- but only until new current is applied. I think what's missing is the watchdog. I'm a bit puzzled how to do that. Since enable and maxcurrent now both write to Power we can't just repeat maxcurrent. Seems both values need be buffered and that value re-written.

@andig
Copy link
Member Author

andig commented Jun 22, 2024

Auch noch die Ist-Leistung (power) mit hinzunehmen als Summe von 1074+1075?

@premultiply Aktuell kann der custom charger noch keine Leistung. Stattdessen wäre ein separates meter am Loadpoint zu verwenden. Das ist nicht ganz offensichtlich und könnten wir hinzu fügen- dann müssten wir uns allerdings fragen, welche der vielen meter APIs wir hier noch anbieten wollen?

Further discussion in #14508

Copy link

@PierreLaule PierreLaule left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Läuft mit den vorgeschlagenen Änderungen

templates/definition/charger/ac-elwa-2.yaml Outdated Show resolved Hide resolved
templates/definition/charger/ac-elwa-2.yaml Outdated Show resolved Hide resolved
templates/definition/charger/ac-elwa-2.yaml Show resolved Hide resolved
templates/definition/charger/ac-elwa-2.yaml Outdated Show resolved Hide resolved
templates/definition/charger/ac-elwa-2.yaml Show resolved Hide resolved
templates/definition/charger/ac-elwa-2.yaml Outdated Show resolved Hide resolved
@andig
Copy link
Member Author

andig commented Jun 29, 2024

@PierreLaule Nice, danke für den Review! Du kannst übrigens direkt den richtigen Code vorschlagen, dann muss ich nur noch "committen"

@andig andig merged commit 2a28de5 into master Jun 29, 2024
6 checks passed
@andig andig deleted the feat/elwa2-charger branch June 29, 2024 13:57
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.

4 participants