Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ossia-max] parameter in poly~ always adds voice # to parameter's name #811

Open
matcham opened this issue Dec 12, 2022 · 8 comments
Open
Assignees

Comments

@matcham
Copy link

matcham commented Dec 12, 2022

if an ossia.parameter is inside a poly~ , voice number is always added at the end of the parameter.
(if the model is outside of the poly~)

ossia.parameter myParam
in the voice 1 of a poly becomes
ossia.parameter myParam.1

here is an example patch
params in poly naming issue.zip

I guess this is an inheritance of how models are auto-renamed inside a poly voice (adding instance number)
but should it be the same for parameters ?
I don't know if it's a feature or a bug... but in my user-case, that's very annoying !

maybe adding an attribute to enable / disable auto-renaming could be useful in this situation ?

@avilleret
Copy link
Contributor

Yes this is a feature, implemented as specified and designed by users.
This feature is there since the beginning, in the age where you couldn't create parameter without a name argument.
I agree with you, in you're particular case, it seems weird.

If I remember correctly, the voice number is added explicitely because we firstly rely on abstraction creation order which was not consistent, in some case voice 30 where instanciated before voice 21 and then some user complain about that behaviour.

Adding another option to enable/disable this feature sounds feasable but I don't have much time these days to work on ossia-max.

@avilleret avilleret removed their assignment Dec 12, 2022
@jln-
Copy link
Contributor

jln- commented Dec 12, 2022

Yes this is a feature, implemented as specified and designed by users. This feature is there since the beginning, in the age where you couldn't create parameter without a name argument.

Well, not totally sure that's what was designed by users, although most likely poorly stated. Given second point (that we couldn't create a parameter without argument), adding parameters in poly~ indeed introduced duplicatas in the namespace thus the need to automatically adding instance numbers. But implicitly there's no such need if there are no duplicatas.

Adding another option to enable/disable this feature sounds feasable but I don't have much time these days to work on ossia-max.

If someone is to work on this at some point, imho Mathieu's case should be made to work rather than adding an attribute.

@avilleret
Copy link
Contributor

well in fact, it is more a side effect of how a feature has been implemented which is no more compatible with some newer advanced capabilities
so let's say it is a bug
but fixing it seems a bit tricky (I can elaborate on that feeling but as I said, don't have much time right now :-( )
as a quick fix, you can use a workaround : declare the parameter outside the poly patcher and put a remote inside the poly~.

@matcham
Copy link
Author

matcham commented Dec 12, 2022

Well, as it is now possible to create a parameter without argument and set the address afterward, it's easy to give it a name based on voice # .
So I would say that this auto-renaming "feature" is not needed anymore.
What about just removing it ?
:-)

@avilleret
Copy link
Contributor

why not but that would break patcher relying on it
but who cares about backward compatibility ?

@matcham
Copy link
Author

matcham commented Dec 12, 2022

I actually have at least 6 different versions of ossia-max for different projects, because lots of things have evolved, old bugs fixed, new one introduced...
so IMHO : let's go !
Backward compatibility will be more an issue when ossia-max will have reached the package-manager ;-)

@matcham
Copy link
Author

matcham commented Dec 12, 2022

(+ that will maybe break old patchers, but it would be very easy to fix)

@avilleret
Copy link
Contributor

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants