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

use latest mavlink definitions for MAV_ODID_ARM_STATUS #295

Merged
merged 1 commit into from
Jan 10, 2023

Conversation

BluemarkInnovations
Copy link

@BluemarkInnovations BluemarkInnovations commented Dec 19, 2022

Copy link
Member

@hendjoshsr71 hendjoshsr71 left a comment

Choose a reason for hiding this comment

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

This is correct to do but we need to merge at the similar time the associated (nonexistent?) AP PR after this goes in to fix the names and move forward the mavlink submodule

AKA the mavlink double shuffle.

@BluemarkInnovations
Copy link
Author

BluemarkInnovations commented Dec 21, 2022

This is correct to do but we need to merge at the similar time the associated (nonexistent?) AP PR after this goes in to fix the names and move forward the mavlink submodule

AKA the mavlink double shuffle.

You are right, I forgot to submit a AP PR, as I initially thought this message was not used by AP.

AP PR: ArduPilot/ardupilot#22460

In general also the OpenDrone ID of Mission Planner need to be updated. I'm in the progress of setting up a build environment for that, but it is a bit time consuming.

@tridge tridge merged commit 5eda0d9 into ArduPilot:master Jan 10, 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.

3 participants