You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Typing a value with its unit ("-6 dB", "500 ms", "50%") into a host's parameter field resets the parameter to its default #17
Hosts call this when a user types into a generic editor or an automation-lane value box. Users naturally type the unit the host displays next to the value, and a typo lands here too.
Observed
On the Gain parameter, whose default is 0 dB:
FromString("-6") -> -6.00 dB
FromString("-6 dB") -> 0.00 dB (reset to default)
FromString("-6dB") -> 0.00 dB (reset to default)
FromString("abc") -> 0.00 dB (reset to default)
Delay Time behaves the same way: "500 ms" jumps to the 250 ms default. So does Mix: "50%" jumps to its default. The user's current setting is silently thrown away, and automation can be written with the wrong value.
Suggested fix
Before parsing, trim whitespace and strip the descriptor's Units suffix (and a trailing %), ignoring case.
On a parse failure, return the parameter's current normalized value instead of DefaultNormalizedValue, so bad input leaves the value unchanged.
Acceptance criteria
Unit tests: "-6 dB", "-6dB" and " -6 " all give -6 dB on Gain. "500 ms" gives 500 ms on Delay Time. "50%" and "50 %" give 50 on Mix.
Priority: Medium. Typing a value with its unit silently resets the parameter to its default, which can write wrong automation. The workaround is to type a bare number.
Duplicates / in progress: No duplicate or open PR.
Next step: Trim the input and strip the descriptor's Units suffix and % case-insensitively. On a parse failure, return the current normalized value rather than the default. Add the listed unit-test rows.
Activity
matt-edmondson commentedon Oct 5, 2026
Triage
EffectsHost.Plugin/EffectAudioParameter.FromStringUnitssuffix and%case-insensitively. On a parse failure, return the current normalized value rather than the default. Add the listed unit-test rows.Generated by Claude Code