add CarState.odometer (Kilometerstand)#3210
Conversation
| _diff = int(_now - _carState.soc_timestamp) | ||
| if _diff > self.general_config.request_interval_charging and\ | ||
| vehicle_update_data.plug_state and\ | ||
| vehicle_update_data.charge_state and\ |
There was a problem hiding this comment.
Warum hast Du das geändert?
There was a problem hiding this comment.
Sorry, gehört eigentlich nicht in diesen PR, aber der war eben offen.
Resultierte aus dieser Diskussion mit einem Tester: https://forum.openwb.de/viewtopic.php?p=140943#p140943
Solange nicht geladen wird erscheint ihm der Warn-Zustand im Status "zu hart".
Ich bin da "leidenschaftslos".
There was a problem hiding this comment.
Wenn zwischen Ladestop und letzter SoC-Berechnung noch geladen wurde, wird das nicht berechnet. Am saubersten wäre es, zu prüfen, ob seit der letzten Berechnung geladen wurde. Also ob vehicle_update_data.imported - last_imported > 0 ist.
There was a problem hiding this comment.
Hmm, wird das nicht schon durch diese Bedingung in Zeile 141 erledigt?
(self.calculated_soc_state.last_imported or vehicle_update_data.imported)
There was a problem hiding this comment.
self.calculated_soc_state.last_imported und vehicle_update_data.imported sind absolute Zählerstände, d.h. in Z. 141 wird geprüft, ob die Zählerstände vorhanden sind, aber nicht ob es eine Differenz gibt. Die wird erst in calc_vehicle_data ausgerechnet.
There was a problem hiding this comment.
OK, verstanden. Ich habe es so eingebaut und getestet. Test sieht gut aus.
Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com>
Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com>
Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com>
Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com>
rleidner
changed the title
2 participants
Add odometer (Kilometerstand) to
UI PR: UI changes for carState.odometer (Km-Stand) openwb-ui-settings#923