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

Make Note duration seems to not retrig #27

Open
Tomavatars opened this issue Nov 14, 2023 · 16 comments
Open

Make Note duration seems to not retrig #27

Tomavatars opened this issue Nov 14, 2023 · 16 comments
Assignees
Labels
bug Something isn't working

Comments

@Tomavatars
Copy link

If the duration is short, then I can hear clicks after a while. If its very long, the clicks never occurs (before I reach the duration limit). I'm using Metasound enveloppes.
It seems that duration is not retrig with new notes.

RNBO_Click1_V1.mp4
RNBO_Click2_V1.mp4
RNBO_Click3_V1.mp4
@x37v
Copy link
Contributor

x37v commented Dec 14, 2023

@Tomavatars are you able to share the patch that you're using for this?
I'm concerned with your use of external envelopes and those potentially not being timed exactly with the MIDI notes coming in

@jinpavg
Copy link
Contributor

jinpavg commented Dec 14, 2023

@Tomavatars would you also mind sending a screenshot of the whole MS graph? It would be helpful to see where the trigger that is patched to MakeNote and the trigger that initiates your audio envelope are.

@Tomavatars
Copy link
Author

@Tomavatars are you able to share the patch that you're using for this? I'm concerned with your use of external envelopes and those potentially not being timed exactly with the MIDI notes coming in

Hi! I'm not very used to share Max patch, I'm a bit of a noob with some Max sharing features. Can you tell me how to share you the patch? I'll send you the file here but perhaps there's a better way to do this?
FM2OP.maxpat.zip

@Tomavatars
Copy link
Author

@Tomavatars would you also mind sending a screenshot of the whole MS graph? It would be helpful to see where the trigger that is patched to MakeNote and the trigger that initiates your audio envelope are.

Here you are:

CleanShot 2023-12-18 at 10 56 48
CleanShot 2023-12-18 at 10 58 16
CleanShot 2023-12-18 at 10 59 16

@x37v
Copy link
Contributor

x37v commented Dec 18, 2023

@Tomavatars I'm suspicious of this part of your patch

image

If the metasound envelope isn't super well timed with rnbo's notion of a note on/off, you might be getting some discontinuities there. This seems likely because you're modulating your amplitude envelope duration but not the duration of your midi notes. You could try adding some smoothing (line~) between your [receive Velocity] and the [+~] and you might benefit from filtering out 0 before you do the [/ 127]

You also might find things a little easier if you move your envelopes into your rnbo patcher, but maybe you have some additional plans with these envelopes (you could always send them out of your rnbo patcher if you need)

@Tomavatars
Copy link
Author

@x37v thank you, that's useful!
Actually, I wanted to use the envelop in Metasound especially for quality of life. I don't like to have a ton of inputs in a node.
But if I could use arrays inputs (like I could have an adsr array) that would be cool. And also because I can also use other envelop technics directly in Metasound.
What do you think of that?

@x37v
Copy link
Contributor

x37v commented Dec 18, 2023

I do think that handling arrays somehow would be nice, maybe inport and outport could use them in place of the list type that you get in RNBO, also I could also see using them to control exposed polyphonic parameters, I think I have that included in an issue somewhere here.

I do think that the clicking you're hearing is not coming from the RNBO work but instead from your patching so, I think it is worth closing this issue once you verify if the smoothing I've indicated resolves it.

@jinpavg
Copy link
Contributor

jinpavg commented Dec 19, 2023

captured this idea as #32

@Tomavatars
Copy link
Author

I do think that handling arrays somehow would be nice, maybe inport and outport could use them in place of the list type that you get in RNBO, also I could also see using them to control exposed polyphonic parameters, I think I have that included in an issue somewhere here.

I do think that the clicking you're hearing is not coming from the RNBO work but instead from your patching so, I think it is worth closing this issue once you verify if the smoothing I've indicated resolves it.

Works like a charm, thanks! I'm still nos very used to Max, sorry for that.

@Tomavatars
Copy link
Author

Tomavatars commented Dec 20, 2023

I still have problems with Make Note duration in another patch that doesn't work in the same fashion. Do I have a problem with my understanding of something?
Here, after the first Make Note trigger, wait for the time of duration to complete, and everything is cut after that (the most interesting part is when it's 1 or 2 second duration)
https://github.com/Cycling74/RNBOMetasound/assets/14031754/6433b1de-efbe-4981-9072-4f852b6c2cf5

Try with random duration, weird behavior.

CleanShot.2023-12-20.at.13.34.35.mp4

@Tomavatars
Copy link
Author

Metasound Graph
CleanShot 2023-12-20 at 13 38 55
My RNBO Patch
SBAMMetasound.maxpat.zip

@x37v
Copy link
Contributor

x37v commented Dec 20, 2023

We talked in discord and it seems like the issue is when you trigger makenote at a rate that is higher than the duration of the note and you don't change the note number, we'd expect to get a note off before a new note on but that doesn't seem to be happening. for instance, if makenote has a duration of 1s and you trigger it at 0.5s, you should effectively get 0.5s notes out, note on.. 0.5s note off then immediately another note on 0.5s note off another note on.. etc

@x37v x37v added the bug Something isn't working label Dec 20, 2023
x37v added a commit that referenced this issue Dec 20, 2023
@x37v
Copy link
Contributor

x37v commented Dec 20, 2023

okay, I pushed a new change to develop @Tomavatars
mind giving that a go?

@Tomavatars
Copy link
Author

okay, I pushed a new change to develop @Tomavatars mind giving that a go?

I'm sorry Alex, it's still not working as expected. It has an erratic behavior instead of constant cutting though.

CleanShot.2023-12-21.at.09.54.39.V1.mp4

@x37v
Copy link
Contributor

x37v commented Jan 4, 2024

@Tomavatars I think we identified a problem with your patch, I'm wondering if you've resolved your issue by updating your patch?

@Tomavatars
Copy link
Author

@x37v the change to my patch didn't fixed the duration issue. I haven't dug in it since the start of the holidays though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants