Skip to content

Support new GPS#151

Merged
onkoe merged 9 commits intoSooner-Rover-Team:mainfrom
onkoe:feat/septentrio_mosaic_go_gps
Feb 17, 2026
Merged

Support new GPS#151
onkoe merged 9 commits intoSooner-Rover-Team:mainfrom
onkoe:feat/septentrio_mosaic_go_gps

Conversation

@onkoe
Copy link
Member

@onkoe onkoe commented Feb 16, 2026

Support the new Septentrio GPS receiver!

Changes

  • Create a serial connection instead of using a UDP connection
  • Parse in Septentrio Binary Format (SBF) over Swift Binary Protocol (SBP)
  • Add passing tests

Checklist

  • Test on the Rover
  • Add tests from earlier this week
  • Add a new test using only a minimal messages configuration

@onkoe
Copy link
Member Author

onkoe commented Feb 16, 2026

blocks inside the bay (NO lock from satellites):

blocks outside the bay (clear view of the sky):

@onkoe onkoe force-pushed the feat/septentrio_mosaic_go_gps branch from d5ef789 to 5effae5 Compare February 16, 2026 04:16
@SoroJetsonOrinNano
Copy link

SoroJetsonOrinNano commented Feb 16, 2026

outside the bay on jetson :)

we're not getting any covariances in the Rust code rn, so wanted to make sure that's not our fault. here's the SBF: outside_bay_feb16.zip

edit: it was our fault; the GPS receiver was misconfigured such that it only reported the most basic GNSS data; nothing covariances.

I got rid of all the extras, leaving just the GNSS data its covariances.

@onkoe onkoe force-pushed the feat/septentrio_mosaic_go_gps branch from 5effae5 to 16cf449 Compare February 16, 2026 17:54
@onkoe onkoe marked this pull request as ready for review February 16, 2026 18:21
@onkoe onkoe force-pushed the feat/septentrio_mosaic_go_gps branch from 16cf449 to 59e072b Compare February 16, 2026 21:03
@onkoe onkoe requested a review from logbasem February 16, 2026 21:18
@onkoe onkoe self-assigned this Feb 16, 2026
@onkoe onkoe added hardware/gps something that's about the Swift GPS src/sensors Has to do w/ the `sensors` package lib/soro_gps test on rover before review labels Feb 16, 2026
This was linked to issues Feb 16, 2026
this includes a real SBF file from outside the bay, checking that the
values in the parser match exactly as expected!
@onkoe onkoe force-pushed the feat/septentrio_mosaic_go_gps branch from 62e9c96 to ee85590 Compare February 16, 2026 21:22
@onkoe onkoe mentioned this pull request Feb 16, 2026
13 tasks
Copy link
Contributor

@logbasem logbasem left a comment

Choose a reason for hiding this comment

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

Looks good !

@onkoe onkoe merged commit 94bf94e into Sooner-Rover-Team:main Feb 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hardware/gps something that's about the Swift GPS lib/soro_gps src/sensors Has to do w/ the `sensors` package test on rover before review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Modify code to support new GPS Get a new GPS receiver

3 participants