docs(g6 dev): clean up stale (Open Question #N) references#54
Merged
Conversation
The "Open Questions" sections in g6_02 + g6_03 were renumbered/pruned during PR #53's restructure, but inline `(Open Question #N)` and `(Open Q #N)` parentheticals in the body were not updated, leaving dangling references. Closes #51. Four updates, all replace dangling refs with what the items actually resolved to: 1. g6_02:85 board-id mechanism — was "(Open Question #4)" which no longer exists. Implementation is the build-time `-DPANEL_REV` flag in panel/platformio.ini (envs pico_v021, pico_v031). Replaced the parenthetical with a direct cite to that file. 2. g6_02:168 LED designators — was "(Open Question #2)" pending KiCad extraction. Per user verification: v0.2 and v0.3 designator tables are identical to v0.1; no separate CSV needed. Replaced the prose accordingly. 3. g6_02:243 schematic → physical pin remapping — was "(Open Q #5)" referencing iorodeo's display.cpp::sch_to_pos_index(). That has been replaced by panel/src/layout.{h,cpp} in LED-Display_G6_Firmware_Panel commit 7c042cd. Updated the cross- reference accordingly. 4. g6_03:61 panel-set ordering divergence — was "Spec ↔ impl divergence D9 in g6_04 — three options to resolve". D9 doesn't exist in g6_04. Inlined the three resolution options directly in the table cell; controller bring-up will pick one. No content was removed; only stale numbering pointers were replaced with what they actually pointed at. The current Open Questions sections in g6_02 (2 items) and g6_03 are unchanged. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This was referenced May 24, 2026
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
1 participant
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.
Summary
Closes #51. Pure-docs cleanup: the "Open Questions" sections in
g6_02+g6_03were renumbered/pruned during PR #53's restructure, but inline(Open Question #N)and(Open Q #N)parentheticals in the body weren't updated, leaving four dangling references.Each is replaced with what the item actually resolved to — no content removed, only stale numbering pointers updated.
Changes
g6_02:85(Open Question #4)-DPANEL_REVbuild flag inpanel/platformio.inig6_02:168pending KiCad-source extraction (Open Question #2)verified identical to v0.1 — no separate CSV needed(per @mbreiser confirmation)g6_02:243sch_to_pos_index() ... (Open Q #5)panel/src/layout.{h,cpp}per firmware commit7c042cdg6_03:61Spec ↔ impl divergence D9 in g6_04 — three options to resolveTest plan
(Open Question/(Open Qparentheticals remain that reference numbers outside the current Q lists