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

Gimbal manager: Add gimbal device flags to low word of gimbal manager flags #1928

Merged

Conversation

olliw42
Copy link
Contributor

@olliw42 olliw42 commented Nov 30, 2022

The lower 16 bits of the gimbal manager flags and cap flags are supposed to coincide the gimbal device flags and cap flags. This PR aligns the manager flags & cap flags with the recent additions to the gimbal device flags & cap flags. I have also added .'s in the descriptions of the gimbal manager flags, as it's so in the descriptions of the gimbal manager cap flags.

@hamishwillee
Copy link
Collaborator

Seems reasonable to me - @julianoes assigned to you because it is gimblish :-)

Copy link
Collaborator

@julianoes julianoes left a comment

Choose a reason for hiding this comment

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

Thanks!

@julianoes julianoes merged commit 2bc3222 into mavlink:master Dec 7, 2022
@dagar
Copy link
Member

dagar commented Dec 7, 2022

This is now failing due to GIMBAL_DEVICE_CAP_FLAGS_SUPPORTS_YAW_IN_EARTH_FRAME, etc duplication. The status check on this PR caught the failure.

https://github.com/mavlink/mavlink/actions/runs/3581910723/jobs/6025516683

Screenshot from 2022-12-07 13-11-20

@olliw42
Copy link
Contributor Author

olliw42 commented Dec 7, 2022

omg ... my fault, sorry guys
=> #1932

TSC21 pushed a commit to Dronecode/air-iop-definitions that referenced this pull request Feb 28, 2023
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.

None yet

4 participants