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

added vtol fixed wing preflight check #12933

Closed
wants to merge 1 commit into from

Conversation

dlwalter
Copy link
Contributor

@dlwalter dlwalter commented Sep 9, 2019

Describe problem solved by the proposed pull request
Some VTOLs do not have the ground clearance to allow arming of the aircraft in fixed wing mode. This pull request creates a parameter COM_ARM_FW_VTOL that can be set to (1) allow arming in fixed wing mode or (2) not allow arming in fixed wing mode. It adds a preflight

Test data / coverage
Tests on Cube (FMUv3) and it correctly prevents arming in fixed wing mode when the parameter is set to 0.

Addresses the following issue:
#12931

@dagar
Copy link
Member

dagar commented Sep 10, 2019

Do we need to allow arming as FW at all? When landed you can always transition to FW immediately (bypassing transition logic).

@dagar
Copy link
Member

dagar commented Sep 10, 2019

Any opinion @RomanBapst or @sfuhrer?

@dlwalter
Copy link
Contributor Author

@dagar I think unless you see a future implementation of a STOVL aircraft I don’t see the need.

We like having the full transition logic implemented during disarm as part of our pilot preflight check so we can confirm the functionality.

@dagar
Copy link
Member

dagar commented Sep 10, 2019

Then unless someone objects I'm in favor of implementing this without the circuit breaker.

@dlwalter
Copy link
Contributor Author

Sounds good. I’ll make a new PR.

@sfuhrer
Copy link
Contributor

sfuhrer commented Sep 11, 2019

Then unless someone objects I'm in favor of implementing this without the circuit breaker.

You mean without any parameter, just always disable arming in FW?
Sometimes while benchtesting it's useful to arm directly in FW, but I guess the benefit from being sure that it doesn't arm unintended. Even would be okay for me to do it without parameter.

Enabling STOVL or proper VTOLs with wheels to take off/land in FW would be cool for the future, one thing missing there now is the land detector to be running also in FW (see #12779). Till we have that disabling arming in FW is fine I'd say.

@dlwalter dlwalter force-pushed the pr-vtol-arm-fixed-wing branch from 56fc30f to 702bd36 Compare September 13, 2019 17:13
@dlwalter
Copy link
Contributor Author

Updated and removed the circuit breaker

@rollysing
Copy link

If you want to default it to not allow to arm in FW is one thing; removing it completely is another.

@stale
Copy link

stale bot commented Dec 12, 2019

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@sfuhrer
Copy link
Contributor

sfuhrer commented Jan 20, 2020

New PR: #13987. From my point of view we should not allow arming in fixed-wing.

@stale stale bot removed the stale label Jan 20, 2020
@sfuhrer
Copy link
Contributor

sfuhrer commented Jan 28, 2020

Close this as #13987 is merged.

@sfuhrer sfuhrer closed this Jan 28, 2020
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