Skip to content

soc_calculation_threshold, range calculation#3153

Merged
LKuemmel merged 14 commits intoopenWB:masterfrom
rleidner:soc_calc_threshold
Mar 11, 2026
Merged

soc_calculation_threshold, range calculation#3153
LKuemmel merged 14 commits intoopenWB:masterfrom
rleidner:soc_calc_threshold

Conversation

@rleidner
Copy link
Collaborator

@rleidner rleidner commented Feb 22, 2026

implements 2 new features:

  • optional calculation when timestamp returned from EV-Server is too old
  • calculation of range together with soc

includes a new field in settings ui: openWB/openwb-ui-settings#908

Copy link
Contributor

@LKuemmel LKuemmel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Die Prüfung, ob der vom Fahrzeug gelieferte SoC aktuell ist, ist ein sinnvolles Feature. Um das UI für den Endbenutzer übersichtlich zu halten, sollte jedoch keine neue Einstellung hinzufgefügt werden, sondern eine sinnvolle Voreinstellung gewählt werden.

Um den aktuellen SoC berechnen zu können, braucht man den letzten bekannten SoC vom Fahrzeug und den zugehörigen Zählerstand vom Ladepunkt. Aktuell stehen bereits der SoC und der letzte Zählerstand als Variable zur Verfügung (last_soc und last_imported). Wenn man länger als den aktuellen Zyklus auf einen SoC vom Fahrzeug warten will, braucht man noch zwei separate Variablen in denen der letzte abgefragte SoC und der zugehörige Zählerstand gespeichert werden.
Um keine großen Sprünge im SoC zu haben und die Implementierung schlank zu halten, würde ich vorschlagen, sobald der SoC-Timestamp älter als das Request-Intervall ist, mit der Berechnung zu starten. Sobald das Auto wieder einen aktuellen SoC liefert, wird ja wieder der SoC aus dem Fahrzeug verwendet.

@rleidner rleidner requested a review from LKuemmel March 10, 2026 20:35
Copy link
Contributor

@LKuemmel LKuemmel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht schon gut aus, jetzt noch der Feinschliff.

@rleidner
Copy link
Collaborator Author

Hallo Lena, der Feinschliff sollte erledigt sein.

@rleidner rleidner requested a review from LKuemmel March 11, 2026 11:02
@LKuemmel LKuemmel merged commit a5d0b1f into openWB:master Mar 11, 2026
1 check passed
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.

3 participants