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

AHRS: added AHRS_OPTIONS and jamming simulator #25676

Merged
merged 3 commits into from
Dec 5, 2023

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Dec 1, 2023

this brings in the separable parts of #24899 to try to progress that work

this includes the AHRS_OPTIONS addition to allow disabling of DCM fallback in plane

Copy link
Contributor

@khancyr khancyr left a comment

Choose a reason for hiding this comment

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

Looks good. I don't understand fully the dcm fallback but the code seems to match to comments

libraries/AP_AHRS/AP_AHRS.cpp Outdated Show resolved Hide resolved
tridge and others added 3 commits December 5, 2023 10:56
this one has been really bad for a while
the first option is to disable DCM fallback on fixed wing. This is
suitable in environments with a high likelyhood of GPS interference
@Ryanf55
Copy link
Collaborator

Ryanf55 commented Dec 5, 2023

We would like @robertlong13 to test this before merge to make sure it meets his needs.

Copy link
Collaborator

@robertlong13 robertlong13 left a comment

Choose a reason for hiding this comment

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

This is perfect. Exactly what I need. I like that you kept it so that it still falls back if the EKF entirely gives up on attitude or vertical position.

Tested with a quick QHover/FBWB flight in RealFlight.

@tridge tridge merged commit 7601a02 into ArduPilot:master Dec 5, 2023
86 of 87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GPS WikiNeeded needs wiki update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants