combine camu.imubu.control and camu.target and simplify camu.select?

[diemoschwarz]

• camu.imubu.control is always followed by camu.target (and needs the same 2nd-to-last inlet connection)
• they could be combined (camu.target inside camu.imubu.control), since camu.imubu.control is used for specifically getting imubu to work
• (camu.target can even be made to work when inadvertently left after camu.imubu.control)

[diemoschwarz]

• looking further, camu.target was also used in camu.select (and camu.voice, etc.)
• if we incorporate camu.target into camu.imubu.control, we can simplify camu.select to have only 2 inlets!!!
  • this would remove the cringeworthy decision which of the first 2 "control" inlet to use
• this means repatching in a few tutorials (3, 4, 5, 13) and examples, tell me if you're working on any of these right now, @einbond @christophertrapani

[christophertrapani]

I'm not working on those at the moment, Diemo. I'm wondering how far into this we want to go, because strictly speaking: there's also an inconsistency with analysis modules. Some tutorial patches use "p corpus-analysis" (see #7) and some use "p descriptor-analysis" (see #12); should these be standardized too (apart from the segmentation tutorials which introduce them)?

…

On Thu, Mar 9, 2023 at 4:45 PM diemo ***@***.***> wrote: - looking further, camu.target was also used in camu.select (and camu.voice, etc.) - if we incorporate camu.target into camu.imubu.control, we can simplify camu.select to have only 2 inlets!!! - this would remove the cringeworthy decision which of the first 2 "control" inlet to use - this means repatching in a few tutorials (3, 4, 5, 13) and examples, tell me if you're working on any of these right now, @einbond <https://github.com/einbond> @christophertrapani <https://github.com/christophertrapani> — Reply to this email directly, view it on GitHub <#76 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIRYDC5ZCSB6DKNEBEHV65TW3H3IXANCNFSM6AAAAAAVTRVKRI> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[einbond]

Diemo, not working on those tutorials so please feel free to make the change! Chris, if we can fix inconsistencies like that then let’s.

…

On Mar 9, 2023, at 5:01 PM, Christopher Trapani ***@***.***> wrote: I'm not working on those at the moment, Diemo. I'm wondering how far into this we want to go, because strictly speaking: there's also an inconsistency with analysis modules. Some tutorial patches use "p corpus-analysis" (see #7) and some use "p descriptor-analysis" (see #12); should these be standardized too (apart from the segmentation tutorials which introduce them)? On Thu, Mar 9, 2023 at 4:45 PM diemo ***@***.***> wrote: > > - looking further, camu.target was also used in camu.select (and > camu.voice, etc.) > - if we incorporate camu.target into camu.imubu.control, we can > simplify camu.select to have only 2 inlets!!! > - this would remove the cringeworthy decision which of the first 2 > "control" inlet to use > - this means repatching in a few tutorials (3, 4, 5, 13) and examples, > tell me if you're working on any of these right now, @einbond > <https://github.com/einbond> @christophertrapani > <https://github.com/christophertrapani> > > — > Reply to this email directly, view it on GitHub > <#76 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AIRYDC5ZCSB6DKNEBEHV65TW3H3IXANCNFSM6AAAAAAVTRVKRI> > . > You are receiving this because you were mentioned.Message ID: > ***@***.***> > — Reply to this email directly, view it on GitHub <#76 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADRCSBLTST2NZUPBAPZWTFTW3H5GBANCNFSM6AAAAAAVTRVKRI>. You are receiving this because you were mentioned.

[diemoschwarz]

some use "p corpus-analysis" and some use "p descriptor-analysis"

Shouldn't all of them use camu.analysis?

[diemoschwarz]

done with BIG commit f6cb8ea

[diemoschwarz]

dang, we forgot to think about backwards compatibility. camu.select is around since oct 2019. Anyone using it will have to repatch...
Alternatively, we could keep the 2 middle inlets and just not use them in the examples (or say it is an "alternative use").

[diemoschwarz]

I fell into the trap myself, having used camu.select in other patches that suddenly stopped working...
I propose to undo the changes but keep the simplified tuts.