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

Copter: Fix payload place bug #27793

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

lthall
Copy link
Contributor

@lthall lthall commented Aug 8, 2024

There is a missing pos_control->update_z_controller();

Before:
image

After:
image

@lthall lthall requested a review from rmackay9 August 8, 2024 06:02
Copy link
Contributor

@rmackay9 rmackay9 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@rmackay9
Copy link
Contributor

rmackay9 commented Aug 8, 2024

I've marked this for next week's dev call but I think we should merge before then because it is a relatively straight forward fix to a relatively serious bug. If any other devs puts an approval on this then I'll likely go ahead and merge.

@peterbarker peterbarker merged commit 16aa2ed into ArduPilot:master Aug 8, 2024
73 checks passed
@rmackay9
Copy link
Contributor

This is included in 4.5.6-beta1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 4.5.6-beta1
Development

Successfully merging this pull request may close these issues.

4 participants