Skip to content

Conversation

@d33psky
Copy link
Contributor

@d33psky d33psky commented Sep 9, 2025

The default (old) GPS units talk at 9600 baud
The newer UBlox-10 GPS series talk at 115200 baud
This PR adds a menu item to select between these 2 speeds and it updates the gpsd configuration in /etc/default/gpsd, and restarts the daemon on change, irrespective of whether UBLOX or classic GPSd is selected so that both will work.

@mrosseel
Copy link
Collaborator

mrosseel commented Sep 9, 2025

hi, just a question - is it a good idea to make the baud rate configurable by the user? Maybe it's better to package this baud rate in with the selection of the GPS chip. Only devs will need to change it to test some things. What do you think?

@d33psky
Copy link
Contributor Author

d33psky commented Sep 9, 2025

Yeah I've thought of that as well.
Ideally we can easily detect all the various units but afaik we do not have that (yet).

The "GPS Type" menu lists

  • GPSD (generic)
  • UBlox

Both can have 9.6k and 115k2 devices.

We could extend it like this :

  • GPSD 9600 (generic)
  • GPSD 115200 (generic)
  • UBlox 9600
  • UBlox 115200

But that looks messy too.

Maybe we should rename the "GPS Baud Rate" menu to "GPS chip" to enumerate the various GPS chipsets ?

  • GT-U7
  • Foxeer-M10Q
  • GEP-M10

Or more generic like:

  • UBlox-6
  • UBlox-10

A chipset list will probably become too long.

@mrosseel
Copy link
Collaborator

yeah it's not clear what the best approach is here, mainly concerned about users changing defaults and getting into trouble. If @brickbots is fine with your original proposal we can start there.

Further thoughts:

  • There's mainly the V1 gps (gpsd 9600) and the V2/V2 gps (ublox 9600), the rest are custom gps'es. Would make it easy for users and still possible for custom gps'es to be activated

@brickbots
Copy link
Owner

Sorry @d33psky I'm late to the party here. I've been thinking about adding an 'Advanced' section under settings that would have GPS and Camera options... ideally with a message on access stating that any purchased PiFinder already has everything set correctly and you can only get into trouble :-)

Would you be willing to add the 'Advanced...' option and put the camera and GPS settings underneath? If it's hidden away a bit I think it's okay having the chip and baud menus!

@d33psky
Copy link
Contributor Author

d33psky commented Sep 28, 2025

Sorry @d33psky I'm late to the party here.

Was indeed beginning to wonder when you'd join ;-)

I've been thinking about adding an 'Advanced' section under settings that would have GPS and Camera options... ideally with a message on access stating that any purchased PiFinder already has everything set correctly and you can only get into trouble :-)

That sounds like a fair grouping. I like it.

Would you be willing to add the 'Advanced...' option and put the camera and GPS settings underneath? If it's hidden away a bit I think it's okay having the chip and baud menus!

Sure. I'll update this PR.

  Settings
  ├── User Pref...
  ├── Chart...
  ├── Camera Exp
  ├── WiFi Mode
  ├── Mount Type
  └── Advanced
      ├── PiFinder Type
      ├── Camera Type
      └── GPS Settings
          ├── GPS Type
          └── GPS Baud Rate

when entering the Advanced submenu a 3 second popop shows that reads:
Options for
DIY PiFinders
@d33psky
Copy link
Contributor Author

d33psky commented Sep 29, 2025

image image image image

@brickbots brickbots merged commit bb214d0 into brickbots:main Oct 8, 2025
1 check failed
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