Repository navigation
Docs: newer trap and failure registers; the stock DELAY reads its whole table run - #1229
Merged
Merged
Conversation
…s the whole table run The vendored TRAPS.md and FAILURE_MODES.md are at upstream 5 Nov 2026 byte for byte, in an eighth docs-only import record that joins the docs-import tests. The stock DELAY reads its coefficient table across the whole zero run 0x400d24d0..0x400d2ce0 at default settings and at boot, so PLACEMENT.md no longer lists it as free, and the hardware traps list it with the page plane and the stock file layer's buffer (trimmed elsewhere so the file does not grow). Co-Authored-By: Claude Opus 5.5 <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.
The vendored trap and failure registers move to the upstream revision of 5 November 2026, byte for byte, in an eighth docs-only import record. The main correction for Modwerk: the stock DELAY reads its coefficient table across the whole zero run
0x400d24d0..0x400d2ce0, not 16 bytes.What changes
sdk/octabam/docs/TRAPS.mdandFAILURE_MODES.md: upstream's files at54200264, unchanged.HARDWARE_MATRIX.mdandMIDI_TRACKS.mdare identical upstream and stay pinned by their earlier records.sdk/imports/octabam-docs-54200264.json: the record (source and vendored SHA-256, review, limitations), added toDOC_IMPORTSinsdk/tests/test_infrastructure_regressions.py.sdk/octabam/docs/remixer/PLACEMENT.md(Modwerk's own copy, no record pins it):0x400d24d0..0x400d2ce0is no longer listed as a free run. The stock DELAY reads all of it as entries 508..1023 of its table: its low-pass index comes from BASE + WIDTH (up to entry 1016), so the default settings read0x400d24d0, and boot reads the last word. The0x400c45b0run is marked as the zero tail of a glyph block, with whether a character reads it unknown.docs/OCTATRACK_HARDWARE_TRAPS.md: the register lists the three entries that reach Modwerk (the DELAY run; the page plane0x46c7e0ealacks windows and popups, which UI captures add throughlcd_view.pybut a barepeekdoes not; a file open through the stock layer owns the caller's buffer). The card-write race paragraph notes it was seen again in emulator runs. The file is over the 40 KB limit, so it was trimmed elsewhere and is 2 bytes smaller.docs/OCTABAM_INFRASTRUCTURE.md: the eighth import's section. Changelog entry.No code, build input, module source or tool pin changes. The card builders' guard for the DELAY run is the separate base-coupled change in #1045.
Review
164f3122…; nothing written): TRAPS.md has no hex string of 8 bytes or more; FAILURE_MODES.md has nine, and no 8-byte window of any occurs in MAIN or in its DSP payloads read as big-endian words. The same scan found planted stock bytes in an in-memory probe.Checks
sdk/tests/test_infrastructure_regressions.py: 22 tests pass.npm run pr:ready: Ready (full check,sdk:checkpassed).What the guides lacked
sdk/octabam/docs/remixer/PLACEMENT.mdlisted the DELAY's table run as free space, and our trap list said nothing about it; both are fixed here. Nothing else.🤖 Generated with Claude Code