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

ExternalAHRS: support variances in backends #28405

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Oct 15, 2024

simplifies backends, allows GCS to see variances in EKF message, and allows for copter EKF failsafe to work
broken out of #24747

Copy link
Contributor

@Georacer Georacer left a comment

Choose a reason for hiding this comment

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

Left some comments.

Has there been any kind of testing?

@amilcarlucas
Copy link
Contributor

Binary Name      Text [B]         Data [B]     BSS (B)      Total Flash Change [B] (%)      Flash Free After PR (B)
---------------  ---------------  -----------  -----------  ----------------------------  -------------------------
arducopter-heli  -136 (-0.0075%)  0 (0.0000%)  0 (0.0000%)  -136 (-0.0075%)                                  142992
antennatracker   -136 (-0.0101%)  0 (0.0000%)  0 (0.0000%)  -136 (-0.0101%)                                  615516
arducopter       -136 (-0.0075%)  0 (0.0000%)  0 (0.0000%)  -136 (-0.0075%)                                  142464
ardurover        -136 (-0.0082%)  0 (0.0000%)  0 (0.0000%)  -136 (-0.0081%)                                  296648
arduplane        -136 (-0.0075%)  0 (0.0000%)  0 (0.0000%)  -136 (-0.0075%)                                  145904
blimp            -136 (-0.0099%)  0 (0.0000%)  0 (0.0000%)  -136 (-0.0099%)                                  593488
ardusub          -136 (-0.0084%)  0 (0.0000%)  0 (0.0000%)  -136 (-0.0084%)                                  352356

re-implement send_status_report in terms of get_variances and support
EKF failsafe with ExternalAHRS
@tridge
Copy link
Contributor Author

tridge commented Oct 19, 2024

@Georacer I had stuffed up the scaling! Fixed up now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants