Skip to content

Implement control pilot interruption retry - #3717

Merged
LKuemmel merged 10 commits into
openWB:masterfrom
seaspotter:cp_interrupt_retry
Aug 4, 2026
Merged

LKuemmel merged 10 commits into
openWB:masterfrom
seaspotter:cp_interrupt_retry

Conversation

@seaspotter

Copy link
Copy Markdown
Collaborator

UI: openWB/openwb-ui-settings#1028

Fälle wie hier #3579 (reply in thread) das Fahrzeuge leider nicht starten obwohl ein Strom permanent vorgegeben ist sind bei manchen Fahrzeug leider häufiger. Abhilfe könnte eine CP Unterbrechung schaffen die einstellbar regelmäßig versucht wird und nicht nur bei Änderung der Ladestromvorgabe wie bisher.

@seaspotter
seaspotter requested a review from LKuemmel July 29, 2026 07:22
@LKuemmel

Copy link
Copy Markdown
Contributor

Muss das Intervall wirklich einstellbar sein? Ich würde gerne vermeiden, den Anwender mit Einstellmöglichkeiten zu überfrachten und bei dieser Einstellung sehe ich keinen Mehrwert im Vergleich zu einer Konstanten. Was meinst Du?

@seaspotter

Copy link
Copy Markdown
Collaborator Author

Hab ich auch darüber nachgedacht, ich habs jetzt auch recht grob gehalten, dass man nur in 30sec Schritten einstellen kann bis max 30min. Die Dauer der CP Unterbrechnung kann man ja auch einstellen, wir können gern aber auch ein festes Intervall nehmen wenn du magst? 3 oder 5min vllt?

@seaspotter

Copy link
Copy Markdown
Collaborator Author

Und die Frage soll es dann an/abwählbar sein vom User oder fest immer versucht werden? Dann würd ichs entsprechend abändern.

@LKuemmel

Copy link
Copy Markdown
Contributor

Dass das Intervall einstellbar ist, ist doch gut so. Wenn man die CP-Unterbrechung immer alle zB 15 Min wiederholt, weckt man jedes Mal das Auto auf, das ist auch nicht gut.
Um zu verdeutlichen, dass man die Funktion auch ausschalten kann, wäre ein Schieberegler für die Zeitspanne ähnlich wie beim Mindeststrom für PV-Laden gut.
Außerdem bitte noch in der update_config das Ev-Template um die Einstellung control_pilot_interruption_retry_interval und die Control Parameter um das timestamp-Topic erweitern, damit die Topics immer konsistent sind.

@seaspotter

Copy link
Copy Markdown
Collaborator Author

update_config ist drin und ich hab im UI jetzt auch auf einen Slider gewechselt der einstellbar von Aus bis 20min geht, ich denke das gibt genug Einstell-Spielraum?

@LKuemmel
LKuemmel merged commit 1501a98 into openWB:master Aug 4, 2026
1 check passed
@seaspotter
seaspotter deleted the cp_interrupt_retry branch August 4, 2026 09:23
ndrsnhs pushed a commit to ndrsnhs/core that referenced this pull request Aug 24, 2026
* Add timestamp_last_cp_retry to control_parameter

Added timestamp_last_cp_retry field to control parameters.

* Add control_pilot_interruption_retry_interval parameter

* Enhance control pilot interruption handling logic

* flake8

* flake8

* Update datastore version to 137 and add upgrade method

* flake8

* Rename cp to control_parameter

* wrong indent

* Refactor check_timestamp call for readability
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.

2 participants