Removed Galileo nav rate change. Slightly reduced Galileo's maximum tracking channel. #597
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The current setting we use is incorrect for a 2 Hz configuration. The update interval is set to 166 Hz and sample for calculation is set to 500 and is outside of the chip's capability. This causes NEO-M8 to fallback to the default 1 Hz update rate and is not really desirable.
According to answer from Ublox forum, we actually do not need to tune down the nav rate when enabling Galileo at all as the chip will automatically scale down as necessary. I have been able to confirm this behavior, that is, the nav rate only goes down when Galileo is being tracked and stays at 10 Hz when no Galileo satellite was used in the calculation (and appears to scale down linearly as the number of Galileo satellite tracked goes up). Adding a toggle switch for Galileo is also no longer necessary as older chip user should receive no negative impact from enabling Galileo with this change. The increased update rate should help AHRS calculations greatly.
This PR also reduces the maximum tracking channel of Galileo from 8 to 5 as there are currently only 11 operational satellite inside the Galileo system and it is not likely to observe more than ~5 satellites at a given point on earth. With this change we free up 3 more tracking channels for GPS/GLONASS/SBAS.
Before:
After: