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

Weidmüller: fix implementation #15261

Open
premultiply opened this issue Aug 5, 2024 · 8 comments
Open

Weidmüller: fix implementation #15261

premultiply opened this issue Aug 5, 2024 · 8 comments
Assignees
Labels
backlog Things to do later devices Specific device support

Comments

@premultiply
Copy link
Member

          Neue Erkenntnisse:

Von einem Weidmüller-MA habe ich eine erste Antwort bekommen, die mich nicht wirklich weiter gebracht, aber beschäftigt hat.

consumed_energy_total (Register 457) wird nur beim Modell "Advanced", welches einen MID Zähler verbaut hat angezeigt.

car_state (Reg. 301) kommt nach Weidmüller Aussage vom Auto; sie haben auf das Timing keinen Einfluss. (Werde ich noch mit einem zu leihenden Oszi nachmessen).
Macht es Sinn hier das Reg 302 charger_state zu verwenden, welches aber wine etwas komplexere Aussage macht?

Reg 702 user_current-limit soll für die Regelung das richtige sein. (obwohl Nichtflüchtig??)

Ich habe jedoch per Zufall noch eine Application-Note gefunden, die hier interessant ist. Wir müssen die WB wohl als Satellit konfigurieren (per WEB Oberfläche oder per Modbus) um sie dann über die Register für das Lastmanagement Register 11000ff zu steuern. Hier gibt es auch das Reg 11052 LCM_ACTUAL_CURRENT_LIMIT, welches flüchtig ist.
Die WB muss dann zyklisch auf Reg. 11050 retriggert werden, was mit dem 30s Zyklus von EVCC ja gehen sollte.

Zum Testen habe ich in meinem ioBroker System etwas Test-Code in Blockly/Javascript geschrieben und mich dabei an eure Nomenklatur angelehnt.

Hier mein Code als TXT Datei. Vielleicht Ideengeber für eigene Überlegungen.
ProgBsp.txt

App Note:
Application Note Einbindung der AC Smart in Lade- Lastmanagement (DE, EN).pdf

Modbus Register als Excel-Datei (für mich strukturierter):
AC-SMART_Modbus-Register.xlsx

Nach seiner Rückkehr aus dem Urlaub hat mir ein Weidmüller MA einen weiteren Kontakt zugesagt.
Meine Themen:
Consumed_energy_total
Flüchtiges register für Phasenumschaltung
Car-State

Ich gebe die Ergebnisse dann weiter.

Originally posted by @tobi119 in #15071 (comment)

@premultiply premultiply self-assigned this Aug 5, 2024
@premultiply premultiply added bug Something isn't working enhancement New feature or request devices Specific device support labels Aug 5, 2024
@andig andig removed the enhancement New feature or request label Aug 7, 2024
@andig
Copy link
Member

andig commented Aug 10, 2024

Was ist hier der "bug"? Fehlerbeschreibung fehlt.

@tobi119
Copy link

tobi119 commented Aug 10, 2024

(1) Fehler: zum Regeln wird ein nichtflüchtiges Register verwendet mit der Folge, dass das EEProm irgendwann sterben wird.

Vor allem verstehe ich nicht warum angeblich die externen Stromvorgaben im Flash gespeichert werden.

Wie @premultiply festgestellt hat erfolgt die Regelung derzeit über ein nichtflüchtiges Register (702).

Nach meiner Sicht ist hier Reg 11052 LCM_ACTUAL_CURRENT_LIMIT zu verwenden, auch um bei Ladepausen auf Null zu regeln.
Reg. 11050 muss dann retriggert werden und die WB auf Satellit konfiguriert sein
Reg 705 für Enable verwenden.

(2) Eine Warning wird derzeit beim Disable generiert, da das Auto erst nach 63 Sekunden von Status "C" in den Status "B" übergeht.
Edit: Ich habe am CP Pin des Typ 2 Steckers nachgemessen: es ist wirklich das Auto und nicht eine Verzögerung in der WB.

(3) Für die Phasenumschaltung wird auch ein nichtflüchtiges Register verwendet, dies wird aber nicht so oft vorkommen.
(in der Hoffnung, dass auf das Register nicht zyklisch geschrieben wird)

(4) "Geladen" (Gesamt-Energie) wird mit Null angezeigt, da Register 457 consumed_energy_total bei den Modellvarianten ECO und VALUE nicht gesetzt wird, sondern nur bei ADVANCED (hier ist ein externer MIDI Zähler verbaut)

Der erste Fehler muss behoben werden, die anderen Punkte machen den Adapter besser/ robuster.
Hie hoffe ich in der kommenden Woche noch eine Aussage von Weidmüller zu erhalten über deren Möglichkeiten.

Für Tests stehe ich gerne zur Verfügung.

@hno2fe
Copy link

hno2fe commented Aug 25, 2024

Hi gibt es schon neu Erkenntnisse ?

@tobi119
Copy link

tobi119 commented Aug 25, 2024

Ich habe gerade heute Weidmüller nochmal angeschrieben bezüglich Stellungnahme.
Wenn da nichts kommt müsste der Punkt (1) zumindest umgesetzt werden. Dann wäre der Adapter einsetzbar.
Für (2) und (4) kann man natürlich in EVCC ein work around programmieren. Für (2) gibt es keine Lösung in der WB.
Punkt (3) sollte unkritisch sein.

@hno2fe
Copy link

hno2fe commented Aug 25, 2024

Vielen Dank für die schnelle Antwort(en) und deinen unermüdlichen Einsatz

Darf man Fragen aus welcher Region du Kommst , ich bin der der nähe von Stuttgart zu Hause
LG Olli

@andig andig removed the bug Something isn't working label Sep 1, 2024
@andig
Copy link
Member

andig commented Sep 1, 2024

Soweit ich das verstehe funktioniert die Implementierung. Gerne PR mit den gewünschten Änderungen- die Registernummern auszutauschen ist ja trivial.

@github-actions github-actions bot added the stale Outdated and ready to close label Sep 8, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 13, 2024
@premultiply premultiply reopened this Sep 14, 2024
@premultiply premultiply removed the stale Outdated and ready to close label Sep 14, 2024
@t1mb0
Copy link

t1mb0 commented Sep 17, 2024

Apologies in advance for swapping to English, confirmed the current integration does 'work' but it's writing to non-volatile memory locations which are not meant to be written cyclically (as per manual). They are for seldom-changing configuration adjustments.

@premultiply premultiply added the backlog Things to do later label Sep 17, 2024
@tobi119
Copy link

tobi119 commented Oct 27, 2024

Nachdem von Weidmüller nicht viele Informationen gekommen sind möchte ich meine Versuche und Lösungsansätze zusammenfassen in anhängendem Dokument.
Für die Stromregelung muss das flüchtige Reg. 11052 LCM_ACTUAL_CURRENT_LIMIT verwendet werden. Dazu muss die WB als Satellit konfiguriert sein und Reg. 11050 muss regelmäßig getriggert werden.
Für die Phasenumschaltung wird, wie bereits implementiert, Reg. 318 verwendet.
Die Gesamtenergie wird von der WB nicht berechnet. Sie könnte nur von EVCC aus der "energy last charging" in Reg. 430 aufsummiert werden.
Für weitere Fragen oder Tests bin ich gern ansprechbar. Meine Implementierung habe ich mit ioBroker getestet.
Grüße aus dem oberen Donautal.

Weidmüller WB_ meine Implementierung.pdf

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

No branches or pull requests

5 participants