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

AP_Mount: added sending of position data to Siyi gimbal #27457

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

rmackay9
Copy link
Contributor

@rmackay9 rmackay9 commented Jul 5, 2024

This replaces PR #25736

This adds support for sending the vehicle's position and velocity to the siyi gimbal which allows the lat, lon and altitude to be recorded in each image via EXIF. This has been tested on real hardware and works. Below is a screen shot from the Siyi manual describing the command structure.

image

Co-authored-by: Randy Mackay <[email protected]>
this will allow for exif tags of position for photos
@tridge tridge merged commit b191869 into ArduPilot:master Jul 9, 2024
92 checks passed
@rmackay9 rmackay9 deleted the tridge-siyi-position branch July 9, 2024 01:59
@timtuxworth
Copy link
Contributor

timtuxworth commented Jul 15, 2024

I tested this on master @rmackay9 . It didn't work for me, all I get is "Mount not healthy" and no recording or pictures. If I roll back to 4.5.4 the Siyi mount works, but of course no position data.

@rmackay9
Copy link
Contributor Author

Hi @timtuxworth, thanks very much for the report. Maybe the older Siyi firmware does not like this new message and fails for some reason. Could you try upgrading the Siyi A8 to the latest firmware from Siyi? I tested using an A8 running Siyi gimbal firmware 0.3.7 and camera ver 0.2.6.

@timtuxworth
Copy link
Contributor

I was running 0.3.7 and 0.2.6

@rmackay9
Copy link
Contributor Author

Hi @timtuxworth, OK, thanks. I wonder if perhaps I was using a beta version of the Siyi firmware and not the standard one. Let me try again.

@timtuxworth
Copy link
Contributor

WIth help from @tridge and @rmackay9 - my setup still had SERIAL1_PROTOCOL = 8 (gimbal) even though it wasn't connected. This meant there were two ports (one serial, one net) that used SERIAL1_PROTOCOL = 8. This worked on 4.5.4 but seems to have confused the new master code. I disabled SERIAL1_PROTOCOL (set to 0) and the Siyi A8 mini camera now works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants