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

EM2GO Home: Workaround for V1.1 current and phase setting #15131

Closed
wants to merge 5 commits into from

Conversation

dniakamal
Copy link
Contributor

We would like to you to consider the following PR as a workaround for the EM2GO Home charger to address the following issues:

@andig andig marked this pull request as draft July 29, 2024 19:03
@andig
Copy link
Member

andig commented Jul 30, 2024

@dniakamal die Workarounds hier würden ja 1:1 auch für die Boxen zuschlagen welche dann eine neue Firmware haben. Der aktuelle Stand funktionier. Können wir daher bitte:

  • die Workarounds nur dann aktivieren wenn es sich um eine alte Firmware handelt
  • falls möglich- gewünschten Phasenzustand aus der Charger Implementierung auslesen statt aus dem Ladepunkt
  • auch die Templates so anpassen, dass die richtigen Fähigkeiten angeboten werden

Insbesondere scheint ja hiermit die Phasenumschaltung auch für die Pro verfügbar zu werden- oder ist das ein Irrtum?

@dniakamal
Copy link
Contributor Author

@andig danke für den Input.

Wir können einen Check einbauen, der einen Stromwert im mA Bereich setzt und diesen dann wieder ausliest, um zu erkennen ob die neue Version auf der Home läuft.
Bspw. Set 10,4A, Read 10A = Alte Firmware, Workarounds nutzen oder Read 10,4A = Neue Firmware, keine Workarounds nutzen.

Alternativ könnte man auch in der kommenden Version ein neues Register für die Firmware Version integrieren, welches wir auslesen können.

Den gewünschten Phasenzustand können wir in der aktuellen Firmware nicht auslesen, da er beim Laden auf 3 zurückspringt. Wir sind daher auf den Wert vom lp angewiesen.

Eine Unterstützung der Phasenumschaltung ist für die Pro Power nicht geplant.

@andig
Copy link
Member

andig commented Jul 31, 2024

Wir können einen Check einbauen, der einen Stromwert im mA Bereich setzt und diesen dann wieder ausliest, um zu erkennen ob die neue Version auf der Home läuft

Gute Idee, das kann bei 0,1A Abweichung beim Start passieren.

Alternativ könnte man auch in der kommenden Version ein neues Register für die Firmware Version integrieren, welches wir auslesen können.

Noch besser, hilft dann aber nur zukünftig.

Den gewünschten Phasenzustand können wir in der aktuellen Firmware nicht auslesen, da er beim Laden auf 3 zurückspringt. Wir sind daher auf den Wert vom lp angewiesen.

Darum geht es nicht. Der Ladepunkt kennt den Wunschzustand. Er korrigiert auch, wenn der LP den falschen Zustand hat. Diese Aufgabe soll auch beim LP bleiben.
Die Frage hier ist: können wir aus der Wallbox den tatsächlichen Zustand auslesen? Der darf ruhig abweichend zum LP Wunschzustand sein- darum kümmert sich der LP.

@andig andig added the devices Specific device support label Jul 31, 2024
@dniakamal
Copy link
Contributor Author

Den gewünschten Phasenzustand können wir in der aktuellen Firmware nicht auslesen, da er beim Laden auf 3 zurückspringt. Wir sind daher auf den Wert vom lp angewiesen.

Darum geht es nicht. Der Ladepunkt kennt den Wunschzustand. Er korrigiert auch, wenn der LP den falschen Zustand hat. Diese Aufgabe soll auch beim LP bleiben. Die Frage hier ist: können wir aus der Wallbox den tatsächlichen Zustand auslesen? Der darf ruhig abweichend zum LP Wunschzustand sein- darum kümmert sich der LP.

Ja wir können den Ist-Zustand der Phasen auslesen und ggf. korrigieren, wenn dieser mit dem Wunschzustand des lp abweicht.

@andig
Copy link
Member

andig commented Aug 1, 2024

Dann bräuchte es:

var _ api.PhaseGetter = (*Em2Go)(nil)

// GetPhases implements the api.PhaseGetter interface
func (c *Em2Go) GetPhases() (int, error) {}

@github-actions github-actions bot added the stale Outdated and ready to close label Aug 8, 2024
@github-actions github-actions bot closed this Aug 13, 2024
@andig
Copy link
Member

andig commented Aug 14, 2024

@dniakamal können wir hier noch irgendwie unterstützen?

@andig
Copy link
Member

andig commented Sep 6, 2024

@dniakamal es gibt weiter Anfragen dazu: #14488 (comment). Können wir hier noch irgendwie unterstützen?

@dniakamal
Copy link
Contributor Author

dniakamal commented Sep 18, 2024

@dniakamal es gibt weiter Anfragen dazu: #14488 (comment). Können wir hier noch irgendwie unterstützen?

Ja bitte, wir sind hier leider nicht weiter gekommen. Der Code wurde getestet und funktioniert einwandfrei, jedoch nur mit der Phaseneinstellung aus dem LP, da der Charger die gewünschten Phasen vor dem Umschalten nicht kennt.

Ich habe dazu einen neuen PR erstellt: #16201

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

Successfully merging this pull request may close these issues.

2 participants