DOC: add SIL parachute ejection integration example (#524) - #1131
Conversation
977177d to
8df7099
Compare
|
Two things and then this can go in. 1. The I fixed that directly on 2. The "Dual path: trigger vs controller callback" section uses a Markdown table inside an .. list-table::
:header-rows: 1
* - Approach
- When it fires
* - ``trigger``
- ...
* - controller callback
- ... |
8df7099 to
dea5438
Compare
dea5438 to
a1ecc9b
Compare
RST has no pipe-table syntax. A line starting with "|" is parsed as a line block, so the "Dual path: trigger vs controller callback" section rendered as a column of ragged lines with the pipes and the "---------" separator showing through, rather than as a table. Sphinx does not warn about this, which is why the docs job stayed green and it went unnoticed. Converted to a list-table, matching the two-column explanatory tables already used in docs/user/flight.rst (:header-rows: 1, :widths: 30 70). Verified by building the page with Sphinx 8.1.3 under -W --keep-going: the section now emits one <table class="docutils"> with a real header row, where before it emitted zero tables and one line-block. No new warnings. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
I pushed the table fix to your branch myself ( What changed: the "Dual path: trigger vs controller callback" section is now a For the record, since Sphinx never warned about this and it would have shipped silently, I built the page both ways with Sphinx 8.1.3 under No new warnings either way, which is exactly why CI stayed green on it. I also scanned the rest of the 245-line file for other Markdown leakage — no fenced code blocks, no
|
Summary
docs/user/sil_parachute_ejection.rstshowing Software-in-the-Loop parachute ejection detection wired throughFlight/ parachute trigger callables.Fixes #524
Test plan
sil_parachute_ejection.rstfor Sphinx link resolution (:doc:,:ref:,:class:)cd docs && make htmlif the docs env is available