Repository navigation
feat(korg): 3-Band-EQ im .all-Slot-Editor (v3.304.0) - #34
Merged
Merged
Conversation
…gleichs (v3.304.0) sampleEqualizer3Band.ts lag mit Tests im Projekt, hatte aber keinen UI-Consumer — dasselbe Muster wie bei der uebrigen Mastering-Kette: die Faehigkeit war da, die Verdrahtung fehlte. Neu korgSlotEq.ts (rein): Formatbruecke auf interleavtes Slot-PCM (wiederverwendet die Bruecke aus korgMatch) plus fuenf Voreinstellungen, die auf das Geraet hin gewaehlt sind: Mumpf raus (300 Hz), Druck (120 Hz fuer Kicks), Hoehen auf (6 kHz, hilft heruntergerechneten Samples), Telefon, Sub weg (unter 80 Hz — Kopfraum, den die Electribe unten nicht wiedergibt). Bewusst KEINE Selbst-Normalisierung: ein EQ, der heimlich den Pegel nachzieht, ist nicht nachvollziehbar. Stattdessen wird Uebersteuerung gemeldet, mit Hinweis auf "Korg Match" — mit eigenem Test, damit das nicht spaeter "hilfsbereit" nachgeruestet wird. 17 Tests (Frequenzwirkung je Preset, Stereo-Kanaltrennung, Clipping-Meldung), tsc clean, 11353 gruen. Co-Authored-By: Claude Fable 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.
Der letzte offene Punkt aus dem ModernKorgManager-Vergleich (
omnitribe/docs/vergleich_modern_korg_manager.md, Punkt ④ — MKMs „Node EQ").Warum
sampleEqualizer3Band.tslag mit Tests im Projekt, hatte aber keinen UI-Consumer — dasselbe Muster wie bei der übrigen Mastering-Kette: die Fähigkeit war da, die Verdrahtung fehlte. Genau das war der Kernbefund des Vergleichs.Was
Neu
korgSlotEq.ts(rein): Formatbrücke auf interleavtes Slot-PCM (wiederverwendetinterleavedToBuffer/bufferToInterleavedauskorgMatch) plus fünf Voreinstellungen. Kein eigenes DSP — die Biquads bleiben im Helfer.Die Voreinstellungen sind auf das Gerät hin gewählt, nicht auf ein Mischpult:
Eine bewusste Entscheidung gegen Bequemlichkeit
Der EQ normalisiert nicht selbst. Ein EQ, der heimlich den Pegel nachzieht, ist nicht nachvollziehbar — man weiß hinterher nicht, was der EQ und was die Automatik getan hat. Stattdessen meldet das Ergebnis
clippedsamt Hinweis, dass „Korg Match" das aufräumt. Mit eigenem Test darauf, damit das nicht später „hilfsbereit" nachgerüstet wird.Tests
17 neue Tests. Geprüft wird nicht die Biquad-Mathematik (die hat eigene Tests), sondern was durch die Anbindung dazukommt:
tscclean, volle Suite grün. Der eine lokale Failer (electribe-import.test.tsPan-Histogramm über die lokalee2s-2016.e2sallpat) ist vorbestehend und skippt auf CI.🤖 Generated with Claude Code