-
Notifications
You must be signed in to change notification settings - Fork 1
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
Send /module/new when meow module changed #11
base: master
Are you sure you want to change the base?
Send /module/new when meow module changed #11
Conversation
added more arguments and updated documentation
Hi @The-Modular-Mind - I would appreciate your feedback on this first draft merge request. Testing it with my TouchOSC template now. |
Hey RJ, sorry I didn't get back to this, I did makes some changes per your requests, I'll clean that up and post a test build today, let's follow up on this after. |
Hey can you try the build from here and lmk: You'll see messages like this:
|
Will do - thanks! Might take a couple of days - will need to change my
TouchOSC template code slightly from my own version of the PR.
I have also just added another PR (on top of my first PR) that implements
my idea of saving client state in the OSC patch config / preset. Seems to
work well locally
4142724
…On Sun, 16 Jul 2023 at 23:00, The-Modular-Mind ***@***.***> wrote:
Hey can you try the build from here and lmk:
https://github.com/The-Modular-Mind/oscelot/releases/tag/latest
You'll see messages like this:
{ address: '/oscelot/moduleMeowMory/start', args: [ 'VCF', 'VCV VCF', 'Voltage-controlled filter', 7 ] }
...
...
{ address: '/oscelot/moduleMeowMory/end', args: 7 }
{ address: '/oscelot/bankMeowMory/start', args: 7 }
...
...
{ address: '/oscelot/bankMeowMory/end', args: 7 }
—
Reply to this email directly, view it on GitHub
<#11 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFFV7JDTMYQDH6BIGT5Y7LXQRQANANCNFSM6AAAAAA2C5TF2Y>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Hi - your test build works well, thanks very much. |
HI - I have done some more testing, and realised that your proposed message is missing the same module name that is sent on the /fader/info message. My TouchOSC script needs that same string to match the info messages to the module:
In the example above, the |
Will fix this in the next build |
Fixes #9
Summary of changes:
/module/new
OSC messages sent when switching meow modules