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

Do not immediately activate AIRMODE when arming with yaw stick arming safety bypass and mc_airmode_type = STICK_CENTER #6531

Closed
avsaase opened this issue Jan 28, 2021 · 4 comments

Comments

@avsaase
Copy link
Member

avsaase commented Jan 28, 2021

Current Behavior

When using mc_airmode_type = STICK_CENTER and arming using the yaw stick override as by nav_extra_arming_safety = ALLOW_BYPASS, AIRMODE tries to yaw the quad. This causes the motors to spin up a lot directly after arming and the quad to take off, which is a safety hazard.

Desired Behavior

When using this combinations of settings, wait until the yaw stick is centered again before activating airmode.

@breadoven
Copy link
Collaborator

I noticed in #6685 that you say you now Arm using the Arm switch flicking method. I had this Bypass Arming issue and changed airmode_type from STICK_CENTER to THROTTLE_THRESHOLD ... which stopped the motors spinning up when you Armed. Wondering why you don't use this which is recommended for smaller MR's. This is the first MR I've built so I'm new to this side of things, still trying to understand most of the MR specific settings.

@avsaase
Copy link
Member Author

avsaase commented Mar 8, 2021

I'm also new to the MR side of the hobby and I initially had problems getting airmode to activate using the THROTTLE_THRESHOLD option (#6532). Now I cannot reproduce the problem anymore so it is possible I didn't raise the throttle enough. Like you I have since switched to THROTTLE_THRESHOLD which indeed solves the problem. The default is still STICK_CENTER, though, so I think the behavior should be changed to only trigger airmode if the sticks were centered at least once after arming and then moved away from the center.

@stale
Copy link

stale bot commented Jan 9, 2022

This issue / pull request has been automatically marked as stale because it has not had any activity in 60 days. The resources of the INAV team are limited, and so we are asking for your help.
This issue / pull request will be closed if no further activity occurs within two weeks.

@b14ckyy
Copy link
Collaborator

b14ckyy commented Mar 20, 2024

#7714 should also fix this issue here I think? @DzikuVx if not, feel free to reopen

@b14ckyy b14ckyy closed this as completed Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants