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

More changes bat indicator #1387

Merged

Conversation

rafaellehmkuhl
Copy link
Member

@rafaellehmkuhl rafaellehmkuhl commented Oct 4, 2024

Kapture.2024-10-04.at.19.39.12.mp4

Fix #1381
Fix #1385

…aneour current and power

Consumed power was being accumulated uncorrectly because of the very low sampling rate, which leads to wrong integrations over time.
@rafaellehmkuhl rafaellehmkuhl force-pushed the more-changes-bat-indicator branch from e56752e to 4bb4f16 Compare October 4, 2024 22:38
Copy link
Contributor

@ES-Alexander ES-Alexander left a comment

Choose a reason for hiding this comment

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

LGTM - nice :-)

@rafaellehmkuhl rafaellehmkuhl merged commit 989a0f4 into bluerobotics:master Oct 6, 2024
10 checks passed
@ES-Alexander ES-Alexander added the docs-needed Change needs to be documented label Nov 11, 2024
@ES-Alexander ES-Alexander added the docs-in-progress Included in an open docs PR label Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-in-progress Included in an open docs PR docs-needed Change needs to be documented
Projects
None yet
2 participants