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

Common: Septentrio page details dual antenna setup #5583

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

rmackay9
Copy link
Contributor

PR ArduPilot/ardupilot#25601 adds support for GPS-for-yaw when using a single Septentrio GPS with two antennas. This is new for 4.5.

This has been tested locally and looks OK to me.

@Hwurzburg
Copy link
Contributor

we will need to add a link to this PR in future-wiki-changes.rst when the code merges

tridge
tridge previously requested changes Nov 28, 2023
common/source/docs/common-gps-septentrio.rst Outdated Show resolved Hide resolved
@rmackay9 rmackay9 dismissed tridge’s stale review November 28, 2023 01:03

addressed tridge's request

@rmackay9 rmackay9 force-pushed the septentrio-dual-antenna branch from 20d7fb7 to e15657e Compare November 28, 2023 01:07
@rmackay9
Copy link
Contributor Author

@Hwurzburg,

I've updated this now including adding a link on the future wiki changes page.

I could not find the Unicore page on our wiki so I haven't added a link to that page.

@Hwurzburg Hwurzburg force-pushed the septentrio-dual-antenna branch from e15657e to 4180338 Compare November 28, 2023 16:50
@Hwurzburg
Copy link
Contributor

Hwurzburg commented Nov 28, 2023

I have separated the link so as to be able to merge and will do so..it should refer to this PR which meres on 4.5 release...I have added the antenna offset info link


See the :ref:`<antenna_offsets>` section for a diagram illustrating the directions of these offsets.

Note that the yaw calculation requires the GPS have an RTK fixed lock and the yaw calculation is only correct while the vehicle is upright so it should not be used on vehicle that spend significant time at extreme lean angles (e.g tail sitters).

Choose a reason for hiding this comment

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

I have been using this board, model mosaic-h, for two years. As far as I know, the yaw output does not require the rtk fixed state, only the satellite status is good and 3D locked.


- :ref:`GPS_TYPE <GPS_TYPE>` = 26 (SBF-DualAntenna)
- :ref:`GPS_MB1_TYPE <GPS_MB1_TYPE>` = 1 (RelativeToCustomBase)
- :ref:`GPS_MB1_OFS_X <GPS_MB1_OFS_X>`, :ref:`GPS_MB1_OFS_Y <GPS_MB1_OFS_Y>` and :ref:`GPS_MB1_OFS_Z <GPS_MB1_OFS_Z>` to match the offset of the main antenna from the second in meters. For example if the second antenna is 50cm to the right of main antenna set :ref:`GPS_MB1_OFS_Y <GPS_MB1_OFS_Y>` = -0.5.

Choose a reason for hiding this comment

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

Is antenna offset now mandatory? There is no need to set the offset when using it before, you only need to set the heading output offset of the board.
In this merged pr, will the firmware automatically configure the heading offset value of the board after setting gps_mb1_ofs_xy?

@Hwurzburg Hwurzburg merged commit 4bd1684 into ArduPilot:master Jan 31, 2024
4 checks passed
@rmackay9 rmackay9 deleted the septentrio-dual-antenna branch February 2, 2024 01:03
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.

4 participants