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

flight_mode_manager: ManualAcceleration support weathervane yaw handler #18038

Merged
merged 1 commit into from
Aug 12, 2021

Conversation

dagar
Copy link
Member

@dagar dagar commented Aug 9, 2021

@dagar dagar force-pushed the pr-manual_acceleration_wv_yaw_handler branch from 76c72b0 to d3be1e0 Compare August 9, 2021 00:07
@dagar
Copy link
Member Author

dagar commented Aug 9, 2021

If this (or a minor variation) works then we can cleanly back port it for v1.12.2.

@VTOLDavid
Copy link
Contributor

Thank you @dagar! I will test it this week

@taileron
Copy link
Contributor

taileron commented Aug 10, 2021

If this (or a minor variation) works then we can cleanly back port it for v1.12.2.

change into 1.12.1 wv now works perfect in position, rtl, mission much thanks @dagar:
https://review.px4.io/plot_app?log=ed48f96c-5447-437c-8105-5df8e4c901a6

@VTOLDavid
Copy link
Contributor

If this (or a minor variation) works then we can cleanly back port it for v1.12.2.

change into 1.12.1 wv now works perfect in position, rtl, mission much thanks @dagar:
https://review.px4.io/plot_app?log=ed48f96c-5447-437c-8105-5df8e4c901a6

Same here, works perfect. Thanks @dagar!

@sfuhrer
Copy link
Contributor

sfuhrer commented Aug 11, 2021

How about extending WV_EN to a bitmask with field one: enable in Auto, field two: enable in Position? As I can imagine users not wanting the WV in Position flight mode, but only in Auto modes.

@VTOLDavid
Copy link
Contributor

How about extending WV_EN to a bitmask with field one: enable in Auto, field two: enable in Position? As I can imagine users not wanting the WV in Position flight mode, but only in Auto modes.

Seems a good option, but why someone would like his VTOL to fight against wind?

@sfuhrer
Copy link
Contributor

sfuhrer commented Aug 12, 2021

How about extending WV_EN to a bitmask with field one: enable in Auto, field two: enable in Position? As I can imagine users not wanting the WV in Position flight mode, but only in Auto modes.

Seems a good option, but why someone would like his VTOL to fight against wind?

Maybe somebody wants to be able to precisely set the yaw of the vehicle manually? In low wind conditions that's no problem. And a fairly knowledgeable pilot probably makes a better job than WV. And I haven't personally tried WV in Pos mode, will be interesting to see how well it works when you have a lot of position setpoint changes.

But it's fine for me to bring this in like this, and follow up with another PR later to make it configurable if we see the need.

@dagar
Copy link
Member Author

dagar commented Aug 12, 2021

I'm going to merge this as is, then cherry-pick for v1.12.2. Then in master we can follow up with adding more granularity to WV_EN.

Thanks everyone.

@dagar dagar merged commit 68dcc25 into master Aug 12, 2021
@dagar dagar deleted the pr-manual_acceleration_wv_yaw_handler branch August 12, 2021 13:58
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.

1.12 Release everything great except Weather Vane
4 participants