ArduPilot / ArduPilot/ardupilot

Copter: parachute release is separate from crash-check allows possibility of parachute not being deployed in crash

Open
#4,702 11 comments 0 reactions 0 assignees View on GitHub
Copter Safety
Dominant language
C++
Stars
15.9k
Forks
21.4k
Avg merge
3d 17h
Merged PRs (30d)
119

Description

The code for the regular crash-check is quite separate from the parachute release check.

They use almost exactly the same logic except that the parachute triggers after 1sec of loss of control while the crash-check triggers after 2 seconds. Also the parachute-check checks that the barometer says we're falling.

The separation of the logic means we're open to the possibility of the crash-check triggering without the chute deploying. Maybe that should be possible because we could imagine a simple flip on take-off (we might not want the parachute to deploy in this case).. but perhaps we should at least combine the logic and possibly delay the crash-check timeout if a parachute is installed (to give time to see if the vehicle is falling).

Contributor guide

Open the contributing guide

Research direction

The issue names separate regular crash-check and parachute-release checks but no files, tests, or entry points. Locate both checks, compare their loss-of-control timeout and falling-state conditions, then clarify whether the intended behavior is combined logic, a delayed crash-check, or another agreed rule before defining completion tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot, robotics
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.