search

mavlink / qgroundcontrol

Cross-platform ground control station for drones (Android, iOS, Mac OS, Linux, Windows)
http://qgroundcontrol.io
3.22k stars 3.56k forks source link

Automatic Base Station Setup Broken for Septentrio Receivers #11621

Closed flyingthingsintothings closed 1 month ago

flyingthingsintothings commented 3 months ago

Expected Behavior

When a Septentrio GNSS receiver is connected to the device, QGroundControl should automatically set it up and forward RTCM corrections over MAVLink to the autopilot vehicle. The setup should happen in a reasonable amount of time, like half a minute.

Current Behavior

QGroundControl detects that a Septentrio receiver is connected as the RTK icon is displayed in the status bar, but it fails to set up the receiver correctly and never sends corrections to the autopilot vehicle.

Steps to Reproduce:

  1. Enable "RTK GPS" under "Application Settings > AutoConnect to the following devices"
  2. Connect a Septentrio GNSS receiver over USB
  3. Wait for the RTK icon to appear in the status bar

System Information

Detailed Description

I also tried v4.3.0 and there it works. I know that two pull requests were merged into the GPSDrivers library recently to try and improve automatic setup of Septentrio GNSS receivers as base station. Only one of the two made it into v4.4.0.

First pull request (included in v4.4.0): https://github.com/PX4/PX4-GPSDrivers/pull/155 Second pull request by me (not yet included in v4.4.0): https://github.com/PX4/PX4-GPSDrivers/pull/157

Log Files and Screenshots

\/

\/

\/

FYI @chiara-septentrio

chiara-septentrio commented 3 months ago

This would probably be fixed by updating the GPS drivers submodule to one of the most recent commit containing the fix (either a41210e or 0b79ec4 should work). I saw you updated the submodule before stable v4.4 @julianoes, could it be possible to do the same for future release or for daily builds ?

julianoes commented 3 months ago

I thought I had backported the bugfixes. If not I will have to do so and create v4.4.1 with it.

chiara-septentrio commented 2 months ago

Would it be possible to have it in daily build or 4.4.1 to make the fix available @julianoes ? Or does some testing need to be done first?

julianoes commented 2 months ago

Daily builds are - unfortunately - off master, and not Stable_V4.4. I'm just waiting on a few more fixes before doing the release.

chiara-septentrio commented 1 month ago

I saw the latest release with the fix, it seems to work fine now. Would it be possible to have the same done for master to ensure the fix is in all future releases?

julianoes commented 1 month ago

@chiara-septentrio thanks for the reminder. This should do it: https://github.com/mavlink/qgroundcontrol/pull/11771.