-
-
Notifications
You must be signed in to change notification settings - Fork 640
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
Ocpp: reduce number of configurable timeouts #15842
Conversation
@@ -48,8 +47,8 @@ func NewOCPPFromConfig(other map[string]interface{}) (api.Charger, error) { | |||
MeterInterval time.Duration | |||
MeterValues string | |||
ConnectTimeout time.Duration // Initial Timeout |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das hier sollten wir wohl auch besser umbenennen, da dies mit dem festen OCPP-Begriff ConnectionTimeOut
Verwechslungsgefahr birgt aber etwas ganz anderes meint:
Interval *from beginning of status: 'Preparing' until incipient Transaction is automatically canceled, due to failure of EV driver to (correctly) insert the charging cable connector(s) into the appropriate socket(s). The Charge Point SHALL go back to the original state, probably: 'Available'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich möchte zu zwei existierenden Timeouts jetzt keinen weiteren einführen der noch neue Verwirrung stiftet. Immer dran denken: dass müssen die Anwender alles nachvollziehen...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das will ich ja auch nicht.
Es ging nur um die Namensgebung, die speziell im OCPP-Umfeld halt etwas verwirrend sein könnte.
Ich glaub in |
Co-authored-by: premultiply <4681172+premultiply@users.noreply.github.com>
Findest Du welche das ist? Konnte auf die Schnelle nix entdecken. |
Hatte das wahrscheinlich verwechselt mit dem hier: Lines 12 to 15 in e3549cf
|
Ich habe message/ meter/ transaction expiry jetzt auch noch weg geworfen. Warum wir auf 1h alte Transaktionen reagieren sollten ist nicht so richtig klar. Wenn die Uhren auseinander laufen haben wir ohnehin schon andere Probleme. |
TODO