See attached file for a random rhythm generator hooked up to a random frequency generator. the former is the focus here.

it's the first patch i have ever made in Pd completely independently. initialize the expression with a value by clicking a message. enter your desired bpm in the number box and hit enter. this begins the metro, which also bangs a random number generator (at top), subsequently banging out a number under 100. the number is compared against three parameters, each corresponding to a note length: half, eighth, sixteenth. depending on the random number's value, it sends the note length multiple (.5 or 2 or 4) to the expression box to be multiplied with the given bpm, thus establishing the metro object's argument for the note.

i'm getting stack overflow. i think it's the connection from the trigger object to the random number generator that is causing the overflow-- but this is an essential part of the build. there is probably a better way of doing this, but shouldn't this work too? i'm down to learn other methods, but to my mind the patch should work well.

let me know what i am missing.

@butter.melting There is not enough information in a screenshot for us to analyse your patch because you have not used triggers everywhere........ but on the good side you prompted me to finally post this..... https://forum.pdpatchrepo.info/topic/13320/welcome-to-the-forum

It is always a good idea to upload the actual patch.... it is much easier for us to read and test.

It is likely that the [t b b] after the metro at the bottom of your patch is asking for another random number before it bangs [random 1000] and so creates a loop and a stack overflow and never actually does bang [random 1000]...... or does eventually when the loop is finally killed by Pd after 1000 overflows.
Try swapping its outlet connections.
David.

@butter.melting There is an infinite loop in your patch. The metro triggers the speed to change and that bangs the metro. The bang goes through and the loop is closed. Maybe it is enough to just set the new speed and let the metro run on its own.

randomseq.pd

@whale-av thanks for the tip about upload the patch. i'll be sure to do that in the future.

@ingox great edits here, thanks. really clean. completely spaced on the moses object.

at first i saw this and thought that the send and receive objects are functioning the same as in my patch, so the loop would still be there. but i understand having the BPM setting and note length-determining section of the patch be separate than that which activates the metro makes this whole thing functional, and cleaner. thanks for the help.

@butter.melting Yeah the sends and receives are just cosmetic.