-
-
Notifications
You must be signed in to change notification settings - Fork 705
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 support for Elli chargers #4207
Conversation
- Add support for defining manual IP addresses for eebus chargers - Add support for defining `hasMeter` - Add support for defining `chargedEnergy`
- Add eebus presets, one with meter and one without - Move ski definition to default params - Add ip default params - Add 2 Elli templates, one for Connect (without meter) and one for Connect (with meter) and Pro
- Include updated zeroconf library - Temporarily using the PRs original repo in mod
Temporarily hardcode the site voltage. This should be taken from site in the future
This fixes the interval setting being ignored and everything going crazy
Mein Ziel ist hier dass es funktioniert. Die EEBUS Implementierung und das momentane "Interface" sind für mich ein dead end. Möchte hier nicht mehr Zeit als notwendig einsetzen. |
Der Elli Charger Connect hat kein eingebautes Messgerät. Man kann dies selbst nachrüsten, dafür gibt es in Foren eine Anleitung. Der Benefit ist dass gemessene Stromstärken pro Phase gemeldet werden. Die Wallbox selbst liefert ansonsten einfach die vorgegebene Stromstärke zurück. Momentan wird über das Template das jedoch deaktiviert und der Anwender sieht keine Ladeleistung. Er könnte "einfach" das andere Template nehmen und würde dann Werte sehen. Die Wallboxen sind alle installiert und ob der Besitzer jetzt noch einen Modbus Zähler mit mehr Daten einsetzt um dann auch Leistung und geladene Energiemenge zu erhalten, ist wohl nicht sehr wahrscheinlich. Die meisten Anwender sind keine DIY und haben wohl mit der Einrichtung von evcc schon genug Schwierigkeiten vor sich. Meine Frage wäre nun: sollte man das im Template der Connect nicht einfach auch "erlauben", dass die theoretischen Werte geliefert werden? @andig @premultiply was meint ihr? |
Auf keinen Fall falls sie das nicht phasenrichtig macht. |
Was meinst du damit? Es werden pro Phase die Stromstärke zurückgegeben, die als Limit gesetzt ist. Es ist ein theoretischer Wert. Wenn das nicht erwünscht ist, bleibt es dann so wie es ist. Aber wie gesagt, der Anwender kann einfach das andere Template wählen und dann passiert genau das beschriebene. Es ist nicht möglich festzustellen ob da ein richtiger Zähler eingebaut ist oder nicht. |
Wir verwenden die Ströme zur Phasenerkennung für den PV Modus. Die müssen stimmen, wenns nur L1 gibt darf auch nur auf L1 was kommen sonst funktioniert die Erkennung nicht. Ich würds lassen wie es ist. |
- Remove sudo from the makefile - Call `sudo make patch-asn1` from the default workflow build job
Bzgl. ASN: jetzt fehlt noch Docker? Ansonsten LGTM:
|
Wieso? Der läuft als Teil des GitHub Workflows, und da werden die Jobs in |
Nein. Der Docker Build läuft in einem Docker Container. |
Ok, was genau soll da jetzt wo rein? Ich kenne diesen Teil nicht. Darin hat man wohl keinen root Zugriff, benötigt man den überhaupt um die go libraries zu patchen? Ich habe hier dazu keine Umgebung um dazu testen und kenne mich damit nicht aus. Evtl. wäre das dann etwas für einen weiteren PR, von jemandem der sich damit auskennt? |
Done |
hasMeter
andchargedEnergy
in the Elli templated marking lacking support of those)ski
definition to default paramsip
to default paramssite
site
interval
)machineid
/plantid
when available