Skip to content
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

Fix soc_timestamp handling for milliseconds #2197

Merged
merged 3 commits into from
Feb 18, 2025

Conversation

MartinRinas
Copy link
Contributor

Adjust the handling of soc_timestamp to convert values from milliseconds to seconds, some SoC modules might return ms.

@MartinRinas
Copy link
Contributor Author

@LKuemmel es gibt noch weitere SoC Provider (z.b. Tesla) welche den Timestamp in ms setzen. Mit diesem PR wird das zentral erkannt und korrigiert. Die Prüfung ob der bekannte SoC neuer ist berücksichtigt das ebenfalls und lässt ein Überschreiben zu wenn der bekannte SoC zu weit in der Zukunft liegt. Wäre gut wenn wir den auch bald übernehmen können bevor noch mehr in dieses Problem laufen und bis ins Jahr 57050 warten müssen bis der SoC wieder akzeptiert wird.

@MartinRinas MartinRinas marked this pull request as draft February 6, 2025 07:34
@MartinRinas MartinRinas marked this pull request as ready for review February 6, 2025 07:43
@benderl
Copy link
Contributor

benderl commented Feb 12, 2025

@MartinRinas
Kannst Du bitte noch eine Warnmeldung in das Log schreiben, dass vom SoC-Modul ein falsch formatierter Zeitstempel empfangen wurde? Letztendlich sollte das ja im Modul bereits korrekt behandelt werden.

@MartinRinas
Copy link
Contributor Author

hab' mir für eine Debugmeldung entschieden, so kann man den Hinweis im Log finden wenn danach gesucht wird. Da ich sonst keine weiteren Einschränkungen erwarte und ein epoch Zeitstempel in ms auch nicht so arg ungewöhnlich erscheint sollte das genügen. Wollte vermeiden dass hier unnötig eine Warnung in der UI erzeugt wird wo ja alles funktioniert.

@MartinRinas
Copy link
Contributor Author

@benderl könnt ihr den Fix mit in den RC aufnehmen? Tronity scheint, zumindest für die PSA Modelle, ebenfalls den timestamp in ms zu liefern weswegen der SoC bis ins Jahr ~55000 nicht übernommen wird. https://forum.openwb.de/viewtopic.php?p=124071#p124071

@benderl benderl requested a review from LKuemmel February 17, 2025 18:16
@LKuemmel LKuemmel merged commit 173137a into openWB:master Feb 18, 2025
1 check passed
@MartinRinas MartinRinas deleted the fix-SoCTimestamp_in_ms branch February 18, 2025 09:18
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