ppooll icon indicating copy to clipboard operation
ppooll copied to clipboard

ll.midi_out varname problems / convert numbered arguments to patcherargs

Open chausch opened this issue 1 year ago • 3 comments

since ll.midi_out has numbered arguments adding a @varname without providing an argument nr 1 & 2 messes up the patch.

i propose changing the patch to use patcherargs 'device' and 'port' instead, so a varname can be applied without problems.

application case: to use multiple named midi outs.

@jpsteccato what do you think? i haven't used ll.midi_out before but would need to make that change in order to make use of it. any conflicts with patches of yours?

chausch avatar Jul 23 '24 14:07 chausch

ok nevermind, i managed to work around the issue, however i have another problem: the addressing via variables of the device & channel menus seems to work going out of ll.midi_out, but not the other way round.

pattr is only sending the value from the first output of umenu, but ll.midi_out seems to expect the device name as a string. possible solution: create a duplicate of the umenu inside of ll.midi_out that helps to convert index integers to name strings.

chausch avatar Jul 23 '24 15:07 chausch

I think I can make the change to use named patcher arguments like you suggested that is backwards compatible with numbered args.

As for the Port and Channel, I don't think you should need to use the index number at all. @chausch If you use umenus with "Store as Symbol" attribute on, does the store and recall of port & channel work for you?

jpsteccato avatar Jul 29 '24 13:07 jpsteccato

thanks @jpsteccato – in the meantime i worked around the issue with a custom implementation for a custom patch, but maybe we should reevaluate the issue & solution and collect important points in the discussion section?

chausch avatar Aug 08 '24 16:08 chausch