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

gps(septentrio): add dual-receiver moving base heading #23123

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

flyingthingsintothings
Copy link
Contributor

Solved Problem

Some users want to use two receivers to compute heading instead of one with two antennas. Two Septentrio receivers can receive the L5 band which makes the setup better in difficult scenarios where GNSS signal reception might be poor.

Fixes /

Solution

  • Add parameter called SEP_MODE for receiver setup selection
  • Set up both receivers differently based on their instance, one as a moving base for corrections and one as a rover

Changelog Entry

For release notes:

Feature Dual-receiver heading computation using moving base
New parameter: SEP_MODE
Documentation: Need to clarify page

Alternatives

/

Test coverage

/

Context

/

Having a generic interface over the GPS drivers makes dedicated
functionality for each driver harder. Move the Septentrio driver into
its own module under the `gnss` driver directory, and let it have its
own parameters for only the functionality it requires. This also helps
with adding new features because they only need to be implemented for
the driver that wants it, simplifying testing.
Add support for two receivers which was also present in the original GPS
drivers. Also fix the reset command which didn't work.
This removes the SBF code from the common GPS drivers which frees up
space to include the Septentrio driver module by default on some boards.
Add back parameters to choose baud rate and automatically detect baud
rate to use in configuration.
Add support for heading computation using two receivers, one as a moving
base and one as a rover calculating the heading based on corrections
from the moving base.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant