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

GNSS: support MAVLink GNSS receiver status information #3347

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

Conversation

flyingthingsintothings
Copy link
Contributor

@flyingthingsintothings flyingthingsintothings commented May 6, 2024

Add support for GNSS status information from MAVLink, which includes authentication, errors and interference status.

Context:

Add support for GNSS status information from MAVLink, which includes
authentication, errors and interference status.
@EosBandi
Copy link
Collaborator

EosBandi commented May 6, 2024

It looks like a custom mavlink implementation, I see not trace this change in the official mavlink library.
If this is the situation it does not have any place in the master branch.

You cannot simply change already existing mavlink messages.

@flyingthingsintothings
Copy link
Contributor Author

I've linked to the change in the MAVLink spec in the original comment. This PR is still closed as it's only for context until the MAVLink change is discussed/merged/changed...

@chiara-septentrio
Copy link

This PR is not up to date with the finalized MAVLink message, a branch called temp-resilience has a more recent version of the support but is untested

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.

3 participants