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

Add relposheading in process fix uavcan driver (AP Periph Dronecan Dual RTK support) #23821

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

dirksavage88
Copy link
Contributor

@dirksavage88 dirksavage88 commented Oct 16, 2024

Solved Problem

Adds support for dual RTK heading via AP Periph, which publishes heading as a distinct message (RelPosHeading) instead of via the ecef position fields.

Fixes #22520

Solution

  • Add sub callback for relposheading within GNSS sensor uavcan driver
  • Process heading information and validity in process fixx method

Changelog Entry

For release notes:

Feature: Add support for AP Periph (dronecan) dual GPS heading setups
New parameter: None
Documentation: Pending doc update PR in PX4 user guide

Alternatives

Create a sub to the sensor gnss relative uORB topic directly in EKF2

Test coverage

  • Tested using dual Holybro ZED-F9P dronecan nodes and pixhawk 6xrt

Context

TODO: post log
Pending doc update PR: PX4/PX4-user_guide#3406

@dirksavage88
Copy link
Contributor Author

@vincentpoont2

@vincentpoont2
Copy link
Member

@dirksavage88 Anything we can assist on our end?

@dirksavage88
Copy link
Contributor Author

@vincentpoont2 I am having issues getting the rover RTK F9P to get Fix type of fixed (6). It only reports fix type of 3 or 4 even after sitting for a while. (Similar issue to this post: https://discuss.ardupilot.org/t/heading-informations-via-moving-baseline-configuration-gnss-antenna-setup/113674/8)

Other than this issue, I have some questions for the maintainers on how to integrate the gps yaw offset correctly, since it is handled differently in PX4 than what ardupilot is doing.

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.

[CAN GPS] Dual F9P Heading Support for CAN GPS (Non-PX4 CAN FW)
2 participants