fix(falcon): gz-plant findings from the v1.117 recording sessions#271
Merged
Conversation
Three fixes surfaced by flying the supervised rotor-out on the REAL gz plant (each measured, not guessed): 1. FDI rate gate 1 → 2 rad/s (falcon-core): the gz plant hovers with an attitude limit-cycle near 1 rad/s roll/pitch (#270), which held the near-level/low-rate gate chronically shut — the FDI lost the race against the runaway detector on a genuine rotor kill (Terminate at kill+0.6 s, motors correctly cut by the new idle-mode logic, 1 m plop). A true tumble is > 3 rad/s so the gate keeps its purpose; false-trip protection is carried by the v1.115 command-aligned residual. With the widened gate the gz rotor-out isolates immediately: kill 12.0 s → touchdown + Disarmed 13.2–15.6 s across runs. Suites: falcon-core 45/45, falcon-sitl-gz 32/32. 2. NavSat datum guard (falcon-sitl-gz physics): the 3 s first-fix wait raced gz topic discovery on a loaded machine (2-in-3 runs), silently leaving the raw 488 m MSL altitude in the estimator (runaway climb or refused takeoff, arrival-order luck). Now waits 15 s, logs the capture loudly, and REFUSES to fly without a datum. 3. Video tooling: camera framed on the flight column (gui-video.config) + annotate-rotorout-video.sh — layered explainer captions (plain-language + technical register) for the release video. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Verify-Filter: (has-tag "rotor-loss")
Three measured fixes from flying the supervised rotor-out landing on the real gz plant (the recording sessions):
🤖 Generated with Claude Code