Repository navigation
Analog Rytm MKII: Main RMS 1.0.0 moves into the library - #1299
Merged
Merged
Conversation
Moves Iced-Cream's Main RMS 1.0.0 (an elemod port of MrScruff's rytm-mods mod, in rytm2_mods) from _submissions/ into modules/main-rms, code unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Compiled in the pinned modwerk-source-tools image from a clean checkout of d93ad9a and exported with package-elemod-recipe.mjs (--check reproduces it). Its .run, sites, symbols and relocations equal rytm2_mods v1.2's release object, apart from the runs that object takes from the stock OS. Adds the import record, release notes, the library list line and TESTING.md's library package section. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The elekloader record pins the package at 100f1d8; elekloader:update --overlay fetched it into the vendored catalogue. The library parity and planBuild tests list it, and the verification record says what was built and what has not run on a unit. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
repeat98
enabled auto-merge
November 7, 2026 14:00
Keeps Ctrl All (#1298) beside Main RMS in the modules README and the library parity test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
repeat98
added a commit
that referenced
this pull request
Nov 7, 2026
Keeps Main RMS (#1299) beside Slice Loop in the modules README and the library parity test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
4 tasks done
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.
Moves Iced-Cream's Main RMS 1.0.0 from
_submissions/(merged in #935) into the library, so it appears on the site once merged. It is the elemod port (rytm2_mods v1.2,afd461d) of MrScruff's rytm-mods0009-main-rms: on the compressor pages the VOL label draws inverted while the main output is louder than -18 dBFS RMS, for gain staging. Version stays 1.0.0 and the code is unchanged. Needs repeat98's first-release approval.Author's OK: Iced-Cream agreed to our doing the move: #1291 (comment)
sdk/analog-rytm-mkii/modules/main-rms/withgit mv. The manifest,build.json, source, include, media, evidence and every credit line are unchanged (author Iced-Cream; MrScruff's original, published in rytm-mods-src with permission; irpina's core). The submission already hadtests.documentation, the tutorial, 1 rytm-emu capture with their record and measured memory, so the manifest needed nothing more. The modules README lists it..elemodfe24c98b433da9c3d3cfd52965ee6696d22d5affcac14452bde53ac69c2b6f4ainsdk/imports/packages/main-rms/, recorded insdk/imports/main-rms-2026-11-07.json, pinned bysdk/imports/elekloader-main-rms-2026-11-07.json(commit100f1d85) and vendored throughelekloader:update -- --overlay. Built bybuild-elemod-isolated.sh(modwerk-source-tools28323550) from a clean checkout ofd93ad9a2(the move alone).package-elemod-recipe.mjs --checkreproduces it, andverify-elemod-release-exports.mjsreproduces all 45 exports..run, sites, symbols and relocations equal rytm2_mods v1.2's release object (cbd69edb…5d54). Where that object takes 2 runs (16 bytes, at most 8 in a row) from the stock OS at link time, this package carries the bytes compiled from the source, as Filter Hz's package does.media/compressor-page.png, rytm-emu).module:doctor's module rules,machine-modules.mjs --base origin/mainand CI's module contract all accept it, so I took no new captures.sdk/imports/rytm2-mods-main-rms-afd461d.jsonnames module-relative paths, so the move changes nothing in it and its hashes stand (sdk:imports:checkgreen).src/community/module-changelogs/analog-rytm-mkii-main-rms.json, one 1.0.0 entry, 3 lines.main-rms1.0.0;digi-build.test.tsplans it beside Filter Hz and Temp Save on the plain base.docs/verification/2026-11-07-rytm-main-rms-library.md.jmpat0x401213cain the audio interrupt and two pointers at0x401e14ecand0x401e14f0in the compressor view. No library module or base variant patches them, and its claims (ui:compressor VOL label drawing,audio:MAIN level read each block) are no other module's.pr:ready's board check says a move out of_submissions/adds nothing that needs a claim, so it ran withMODWERK_BOARD=off, as for Env Reset Amp (Analog Rytm MKII: Env Reset Amp 1.0.0 moves into the library #1297).Links: #935, #1297. Merge with a merge commit, not a squash, so the pinned commit
100f1d85stays onmain. If I mergemaininto this branch to clear a conflict, the pin stays in the branch history and the same applies.What the guides lacked: nothing wrong. One gap, left out of this module release:
ELEMOD_SOURCE_BUILDS.mddoes not say that an upstream release object may take runs from the stock OS (stockparts) while Modwerk's export carries the compiled bytes, so comparing an export with the author's object means comparing the hex parts at their offsets.Validation commands and results:
npm run module:doctor -- main-rms: manifest, category guide, browser builder and module rules greenMODWERK_BOARD=off npm run pr:ready: Readynode scripts/package-elemod-recipe.mjs … --check,node scripts/verify-elemod-release-exports.mjs <build>: reproduced (45 exports)npx vitest run src/devices/library-parity.test.ts src/engine/elekloader/digi-build.test.ts: 51 passed🤖 Generated with Claude Code