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

Fixed update frequency #7

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

fetzu
Copy link
Collaborator

@fetzu fetzu commented Feb 15, 2022

Aligns the update frequency to Iternio's recommendations.

See #5 (comment) for details.

- Aligned API send frequencies according to Iterino's/ABRP's Telemetry API documentation recommendations (https://documenter.getpostman.com/view/7396339/SWTK5a8w): 1 second for driving car, 6 seconds for charging car, 30 seconds for idle/parked/asleep/offline car.
- Adapted the printed messages to reflect the changes + made the output cleaner/clearer.
- Fixed issue with print output leading to incorrect concatenation. Ooops.
Added {{current_year}} to the copyright notice. Happy 2022 everybody :) !
@fetzu fetzu requested a review from letienne February 15, 2022 17:04
@fetzu fetzu closed this Feb 15, 2022
@fetzu fetzu deleted the fixupdatefrequency branch February 15, 2022 17:06
@fetzu fetzu restored the fixupdatefrequency branch February 15, 2022 17:06
@fetzu fetzu reopened this Feb 15, 2022
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.

1 participant