Setting pan() to invalid values can cause loud noise
gaborveszpremi opened this issue · 1 comments
This is definitely a defect of the firmware of the device (Ui12), not an error in soundcraft-ui, but soundcraft-ui could fix this...
The issue: If I send too low values (for example: -0.7) or too high values (like 50) to pan() function on a channel, the web UI pulls the pan button out of the interface, so the device doesn't protect itself from too low/high pan values. Besides the fact that it's funny, too high values cause extremely loud noise on master output. I think, pan() function should check (and fix) the value (min: 0, max: 1) before sending it to the device. It could avoid destroying a speaker with an invalid pan value...
Oops, thanks for reporting! We should definitely fix that, what a weird bug. 😆 For the fader levels there's already some sanitizing in place and I will adopt this for PAN too.