Skip to content

Fix: Keine CP-Unterbrechung nach Phasenumschaltung#3328

Open
s0170071 wants to merge 1 commit intoopenWB:masterfrom
s0170071:fix/no-cp-interruption-after-phase-switch
Open

Fix: Keine CP-Unterbrechung nach Phasenumschaltung#3328
s0170071 wants to merge 1 commit intoopenWB:masterfrom
s0170071:fix/no-cp-interruption-after-phase-switch

Conversation

@s0170071
Copy link
Copy Markdown

Problem

Nach einer Phasenumschaltung erkennt initiate_control_pilot_interruption() den Uebergang von 0A auf den Sollstrom als neuen Ladestart und loest eine unnoetige CP-Unterbrechung aus. Das fuehrt dazu, dass der EVSE-Strom kurz gesetzt (z.B. 6,9A), sofort wieder auf 0A gestoppt und dann die CP-Leitung getoggled wird, bevor die Ladung tatsaechlich startet.

Auswirkung

Nach jeder Phasenumschaltung wird eine ueberfluessige Stopp/Start-Sequenz durchgefuehrt:

  • EVSE set_current 6,9A (Ladung startet kurz)
  • EVSE set_current 0A (sofort gestoppt)
  • CP-Signal ON (Unterbrechung)
  • CP-Signal OFF
  • EVSE set_current 6,9A (Ladung startet erneut)

Dies verzoegert den Ladestart um ca. 10s und erzeugt unnoetige EVSE-Toggles, die die Relay-Loop-Erkennung ausloesen koennen.

Ursache

In process_algorithm_results() wird nach dem Ende des phase_switch-Threads der Zustand auf WAIT_FOR_USING_PHASES gesetzt. Im selben Zyklus berechnet der Algorithmus einen Sollstrom (z.B. 6,9A). initiate_control_pilot_interruption() sieht current_prev=0, current=6,9 und interpretiert das als neuen Ladestart — obwohl der Uebergang von 0A auf Sollstrom nur durch die gerade abgeschlossene Phasenumschaltung bedingt ist.

Loesung

In initiate_control_pilot_interruption() wird der Zustand WAIT_FOR_USING_PHASES als Ausschluss-Kriterium geprueft. Wenn der Ladepunkt gerade aus einer Phasenumschaltung kommt, wird keine CP-Unterbrechung ausgeloest, da perform_phase_switch() die CP-Leitung bereits selbst umschaltet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant