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

AP_BattMonitor: fixed battery percentage with aux info #25627

Merged
merged 1 commit into from
Nov 26, 2023

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Nov 24, 2023

when we have aux battery information we had assumed the CAN device would provide the battery remaining percentage. We should obey the "do not use CAN SoC" with or without an AUX message

This fixes CAN battery monitors with a cell monitor

when we have aux battery information we had assumed the CAN device
would provide the battery remaining percentage. We should obey the "do
not use CAN SoC" with or without an AUX message

This fixes CAN battery monitors with a cell monitor
@tridge tridge added the BUG label Nov 24, 2023
@tridge tridge requested a review from IamPete1 November 24, 2023 21:32
Copy link
Contributor

@magicrub magicrub left a comment

Choose a reason for hiding this comment

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

Lgtm

@tridge tridge merged commit 11fece2 into ArduPilot:Copter-4.4 Nov 26, 2023
@rmackay9
Copy link
Contributor

rmackay9 commented Dec 4, 2023

This is included in Copter-4.4.4-beta1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 4.4.4-rc1
Development

Successfully merging this pull request may close these issues.

3 participants