sfztools / sfizz-ui

SFZ based sampler, providing AU / LV2 / PD / VST3 plugins using the sfizz library
https://sfztools.github.io/sfizz/
BSD 2-Clause "Simplified" License
44 stars 9 forks source link

Sustain pedal issues in the LV2 plugin, and more general MIDI bindings questions #49

Closed paulfd closed 3 years ago

paulfd commented 3 years ago

The problem is as follows, please tell me if I miss anything:

One immediate impact which is very apparent for users is that by default, sfizz-lv2 does not respond to the sustain pedal, which I think is not a good user experience. We can probably wait for the host to implement the midi bindings eventually, if ever, and just require users to manually map any other sfizz's "CCs" to actual controllers if they would want to, but sustain and sostenuto should probably be treated differently. I propose the following:

cc @jpcima @alcomposer

paulfd commented 3 years ago

Another thing I came across testing things out: Reaper cannot bind lv2:Parameters to midi CC as of version 6.32. They do not appear in the parameter list. Also, Ardour doesn't update the CC parameters touched from the plugin UI in the generic UI.

jpcima commented 3 years ago

Let's just do the PSA perhaps? back when we requested the general opinion, PSA was adviced against and MIDI bindings suggested instead. Following the moddevices discussions, I recall we went full circle on this topic, only to go back to the old way being "map it yourself at plugin-side". (unless I'm mistaken, then correct me)

paulfd commented 3 years ago

If we do the PSA, do we have a way in LV2 to properly update the UI ?

jpcima commented 3 years ago

If we do the PSA, do we have a way in LV2 to properly update the UI ?

MIDI and PSA updates the controls here, in editor and generic UI. Is there an issue on your side?

paulfd commented 3 years ago

Hm no no problem for me. If you're OK with this then we can do that (at least enable by default). I wonder still if it's not a good thing to "hide" the sustain CC from the control page. Or at least make it consistent: for now it will not appear by default, even though regions may be sustained, unless there are some locc/hicc or start_locc/start_hicc switches that e.g. piano libraries do.