looks like a good start. look also to [patcher] so you can encapsulate all the stuff below your buttons/sliders. Makes for a much cleaner patch, and you can use [patcherargs] to set the note values within the subpatch, or set them elsewhere with [preset], [pattr], etc). Also you can use [send] and [receive] to send all your values to one [noteout]. This is helpful because if you want to record the notes with [seq] you can tie it in at the main "out" point.
All words in brackets above are objects, read the help files and documentation for everything as you go, it’s WELL worth the time—you’ll get things done much quicker, and even more, you’ll find new things you can do that you didn’t plan on…!