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_NavEKF3: added two more EK3_OPTION bits #28750

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

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Nov 27, 2024

these allow for easier GPS-denied testing of EKF3, and allows for replay testing without the SetLatLng code being active.

Note that the gaps in the bitmask are for pending range beacon features, we don't want to fill in the gaps so that existing replay logs for that feature remain valid

these allow for easier GPS-denied testing of EKF3, and allows for
replay testing without the SetLatLng code being active.

Note that the gaps in the bitmask are for pending range beacon
features, we don't want to fill in the gaps so that existing replay
logs for that feature remain valid
@rmackay9
Copy link
Contributor

I wonder why we couldn't just use the EK3 source switching feature? there are 3 slots and you can setup one to not use GPS

@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Nov 27, 2024
@tridge
Copy link
Contributor Author

tridge commented Nov 27, 2024

I wonder why we couldn't just use the EK3 source switching feature? there are 3 slots and you can setup one to not use GPS

if this was being used for end users, yes, but as a development option to test the behaviour of EKF3 under GPS denied environments it doesn't produce equivalent behaviour.

@rmackay9
Copy link
Contributor

@tridge,

This seems like a very rare edge case so I wonder why we need a parameter to set it. How about an auxiliary switch and/or a mavlink command?

@priseborough
Copy link
Contributor

@tridge,

This seems like a very rare edge case so I wonder why we need a parameter to set it. How about an auxiliary switch and/or a mavlink command?

With parameters it is possible to change the behaviour when doing tuning and comparison of processing options via EKF replay.

@rmackay9
Copy link
Contributor

Hi @priseborough,

OK, but can't auxiliary switches also be changed easily as part of tuning? I think the recording of the EKF state in the DAL is separate from how the user configures the feature.

Sorry, I don't mean to be difficult but I saw another one of those usability complaints on our YouTube channel and it reminded me that we need to be careful when we expose additional complexity to our users.

Copy link
Contributor

@rmackay9 rmackay9 left a comment

Choose a reason for hiding this comment

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

sorry, I'm adding a "request changes" because I just want to be sure we're thinking through the UI exposed to the user

@tridge
Copy link
Contributor Author

tridge commented Nov 28, 2024

OK, but can't auxiliary switches also be changed easily as part of tuning? I think the recording of the EKF state in the DAL is separate from how the user configures the feature.

there are really 2 parts to this PR:

  • make it possible via an aux switch to disable GPS in EKF3 but not in EKF2, while leaving the src selection parameters alone. This is essential for doing GPS-denied flight testing where you fly on EKF2 but run EKF3 in GPS denied mode
  • make it possible to set this state right from startup, to guarantee that the EKF3 code has never ingested a GPS sample until you move the aux switch to enable GPS in EKF3. We don't (as far as I am aware) have a way to set the initial state of an aux switch. The EK3 code initialises pretty early, often well before the user has properly connected to the vehicle to trigger an AUX switch

These features are being used as part of the testing of the beacon GPS-denied functionality, but Paul and I thought we should PR them as they are useful capabilities for anyone else trying to do GPS denied testing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EKF WikiNeeded needs wiki update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants