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: Performance improvment for U-Blox hardware #2574

Merged
merged 7 commits into from
Jul 2, 2023

Conversation

D4rk4
Copy link
Contributor

@D4rk4 D4rk4 commented Jun 25, 2023

In this Pull Request, I propose improvements to our devices' Global Navigation Satellite System (GNSS) performance by adapting the navigation engine's configuration.

Here are the key changes:

  • Switching to GPS + SBAS (Satellite-Based Augmentation System) + GLONASS for enhanced stability. By leveraging multiple satellite systems, we can ensure more robust satellite coverage and mitigate the effects of signal blockages or interferences that might occur in certain environments.
  • Enabling the AssistNow Autonomous feature. This allows the GNSS module to predict satellite positions based on previously collected ephemeris data, which in turn helps to reduce the Time-To-First-Fix (TTFF). It offers better performance in weak signal conditions and helps to achieve a faster startup.
  • Turning on the anti-jamming feature. This adds an extra layer of protection to our GNSS module by making it more resilient to intentional or unintentional radio-frequency interferences, hence ensuring a consistent GNSS performance.
  • Making sure the new configuration persists across reboots.

By integrating these improvements, I am confident that the GNSS performance of our devices will be significantly enhanced, leading to better stability, faster TTFF, and more reliable functionality under varying signal conditions.

Please review these changes and provide your feedback. These adjustments will greatly contribute to the system's performance and reliability.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 26, 2023

🤖 Pull request artifacts

file commit
pr2574-firmware-2.1.18.988e94f.zip 988e94f

thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 26, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 26, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 26, 2023
@charminULTRA
Copy link
Sponsor Contributor

charminULTRA commented Jun 27, 2023

Did you make sure to use Trunk? That was causing my checks to fail because of formatting issues and therefore build failures.

thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 27, 2023
Copy link
Sponsor Member

@caveman99 caveman99 left a comment

Choose a reason for hiding this comment

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

Great PR, if we can clean it up a bit it can go in!

src/gps/GPS.cpp Show resolved Hide resolved
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 27, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 27, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 29, 2023
@D4rk4 D4rk4 requested a review from caveman99 July 1, 2023 18:07
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jul 1, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jul 1, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Jul 1, 2023
@thebentern thebentern merged commit 4ef61f0 into meshtastic:master Jul 2, 2023
46 checks passed
@kokroo
Copy link
Contributor

kokroo commented Sep 16, 2023

@D4rk4 Which Ublox chips does it work for exactly? Were you able to notice any performance improvements in the real world?

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

6 participants