Im very new to Pure Data so keep answers super simple.

Quick rundown of my intended patch. Trying to make a sequencer subpatch. The first inlet sends in a counting number as a clock, second inlet sends in the amount of beats (modulus). Then the sliders and number set the pitch, and the toggles set the gates/bangs. The horizontal slider is just for decoration. So, bang the pitch slider on every bang from the toggle, it sends to a number, which sends to mtof and then into the first outlet. The second outlet sends out the bangs which are turned on/off by the toggle. The video shows my issues. How can I get this to work?

Also, any advice on making the patch tidier?

@Oscillcreate Nevermind, I found the fix. I was having a stack overflow error due to me running the number output back into the slider input. Deleted that connection and now all is well. Only issue is the number can not be changed with the slider moving along with it now. Any advice on this?

@Oscillcreate You can set a slider using the set message [set $1( in place of the direct feedback, but I am not sure that will do what you want as the counter is still running.
If you need more help then please upload the patch using the up arrow above the box you are typing into.

To keep patches tidy you can use sub-patches to hide parts, or if you have a part that you are copying many times abstractions are great.... especially if you change something as you can change them all at once.
However it is a bit of a learning curve to start using abstractions effectively...... https://forum.pdpatchrepo.info/topic/9774/pure-data-noob

You can also use messages via [send] and [receive] instead of cords, and GUI's like the slider can have send and receive addresses assigned in their "properties" page.
David.