-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add configuration of Beidou and Glonass to M8, M9 and M10 gps modules #9124
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
Conversation
u-center does not show changes (also does not show changes without these changes)
Split out from main ublox file for easier unit testing
Also fix #9123 which affects M8+.
.github/workflows/ci.yml
Outdated
- name: Install dependencies | ||
run: | | ||
brew update | ||
#brew update |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build was failing on dependencies. brew update would reinstall a lot of things and wastes a lot of time.
Commenting it out make the sitl build for mac much faster
I am looking into the test failures. Tests pass on gcc 12 in debian sid, fail on gcc 11 in ubuntu-latest. (Fixed. Unit tests for the win!) |
Add some GPS info to status on the CLI
|
Add protocol vesion parsing
The documentation often refers to messages being obsolete by protocol version, and not by GPS model.
Ublox8 appears to work as well as 10. |
Add configuration of Beidou and Glonass to M8, M9 and M10 gps modules
…sion-check Fix version check and apply GNSS configuration for M8+ gps Should address Galileo setting for M8 and M9 GPS in iNavFlight#9123. M10 requires additional changes in iNavFlight#9124. Just splitting the bug fix from iNavFlight#9124 since it probably requires more testing.
…sion-check Fix version check and apply GNSS configuration for M8+ gps Should address Galileo setting for M8 and M9 GPS in iNavFlight#9123. M10 requires additional changes in iNavFlight#9124. Just splitting the bug fix from iNavFlight#9124 since it probably requires more testing.
Uh oh!
There was an error while loading. Please reload this page.