PX4:main
← flyingthingsintothings:pr-extract-septentrio-driver
opened 09:25AM - 19 Mar 24 UTC
<!--
Thank you for your contribution!
Get early feedback through
- Dronec…ode Discord: https://discord.gg/dronecode
- PX4 Discuss: http://discuss.px4.io/
- opening a draft pr and sharing the link
-->
### Solved Problem
Making changes to the PX4 GPS drivers was hard because new features needed to be implemented for every driver. This isn't possible for most people as they don't have access to the hardware required for testing. During the previous PX4 community Q&A on Discord (2024-03-13), I discussed with @dagar what the options were to simplify this. The solution that was proposed was to separate all the drivers as they all want to specialize more than originally planned. This would allow the drivers to have their own parameters and implement features more rapidly. It would also simplify testing, as people who want to implement a new feature only need access to the hardware for a single driver.
Fixes \/
### Solution
- Extract the Septentrio GPS driver out of the submodule into its own directory, included in the PX4 autopilot repository.
### Changelog Entry
For release notes:
```
Feature/Bugfix XYZ
New parameter: XYZ_Z
Documentation: Need to clarify page ... / done, read docs.px4.io/...
```
### Alternatives
We could also ...
### Test coverage
- Unit/integration test: ...
- Simulation/hardware testing logs: https://review.px4.io/
### Manual Testing
- Pixhawk 6C & mosaic-go heading:
- Automatic GNSS receiver configuration works
- Position shows up in QGroundControl
- Number of reported satellites is correct
- Corrections from base station, forwarded by QGroundControl work
- Automatic startup of driver works
- Connection to receiver is stable and doesn't drop
- `status` command works
- Driver start and stop work
### Context
- [Forum post about developer call where this was discussed](https://discuss.px4.io/t/px4-community-q-a-march-13-2024/37190)
- [Original idea to split up driver](https://github.com/PX4/PX4-Autopilot/pull/22874#issuecomment-1992516897)
- [Teseo GPS driver used as reference](https://github.com/AlexKlimaj/PX4-Autopilot/tree/60a14ce240a09855e4ddf5e9690ca20a46e34aa2/src/drivers/teseo_gps)
- Other drivers in the PX4 autopilot repository were used as a guide (barometer, distance sensor...) because they are also split into separate drivers.