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

Detect launch on gps speed #6726

Merged
merged 4 commits into from
Mar 23, 2021

Conversation

avsaase
Copy link
Member

@avsaase avsaase commented Mar 19, 2021

There have been a couple of reports (here and here, and I believe also one in the Telegram group) where launch was not detected but the configured idle throttle was sufficient to sustain flight. When the throttle is raised but before launch is detected, you cannot exit auto launch by wiggling the stick so if you don't think of putting the throttle back to zero you'll have zero control.

This PR adds an explicit check on the gps speed to detect launch, which should make it impossible for this to happen.

@breadoven
Copy link
Collaborator

A benefit of #6389 perhaps ... it allows you to exit the launch by waggling the stick because the throttle is already low although it would then need the throttle raising to get the motor to rev but at least you regain full control. It also allows you to do this when Launch is selected by switch rather than as a feature unlike the current method which only allows the launch to be cancelled by stick waggle if it's a feature. If it's switched launch you can only exit by switching it off although in that case you wouldn't need to move the throttle low or waggle sticks ... just find the switch quickly.

@avsaase
Copy link
Member Author

avsaase commented Mar 20, 2021

Yes but the problem in these two videos is that they had significant idle throttle configured, which obscured the fact that the launch was not detected. Without idle throttle, the plane would have just crashed a few meters in front of them like any normal failed launch.

I think #6389 will not be very popular with those using idle throttle because it immediately spins up the motor when you arm.

@breadoven
Copy link
Collaborator

And the difference between spinning up the motor on arming and when you nudge the throttle stick above min_check is ?

I'm actually interested to understand why people arm when they aren't about to do exactly what they do when they raise the throttle stick before launching. Are they setting up a ground station perhaps or some other gear ... although I can't see why that would require the craft to be armed. Is there anything that requires arming other than immediately before you launch ?

Like the moisture cured Gorilla glue and water based PVA trick in Adam FPV's video. Would have come in useful fixing the AR Sonic wing after it crashed due to the motor mount unsticking itself and falling off.

@b14ckyy
Copy link
Collaborator

b14ckyy commented Mar 20, 2021

And the difference between spinning up the motor on arming and when you nudge the throttle stick above min_check is ?

Double safety. If you accidently bump the arm switch while throttle is low, it won't suddenly spin in your arm or face. And if the throttle is high it will refuse to arm anyway. Spinning up the motor on arm immediately will take away a full safety layer.

@breadoven
Copy link
Collaborator

And the difference between spinning up the motor on arming and when you nudge the throttle stick above min_check is ?

Double safety. If you accidently bump the arm switch while throttle is low, it won't suddenly spin in your arm or face. And if the throttle is high it will refuse to arm anyway. Spinning up the motor on arm immediately will take away a full safety layer.

Which is why it's an option. I don't use launch idle throttle but if I did I may still prefer to use this method depending on how I assess the risk.

@avsaase
Copy link
Member Author

avsaase commented Mar 20, 2021

Guys let's keep the discussion about arming and safety practices in #6389. This PR is just about launch detection.

@digitalentity digitalentity added this to the 2.7 milestone Mar 23, 2021
@digitalentity digitalentity merged commit a0b243c into iNavFlight:master Mar 23, 2021
@digitalentity
Copy link
Member

digitalentity commented Mar 23, 2021

Looks good now, thanks!

@avsaase avsaase deleted the avs-detect-launch-gps-speed branch March 23, 2021 08:43
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.

4 participants