-
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
fix? #4
Comments
Hi Zach. Does it work properly for you with this fix? |
@ofZach I was reading the code and it seems like you need to load an xml file which contains which midi message info must be linked to a certain parameter. |
thanks! recoded is here to be honest, I haven't really followed much, I just wanted to try to get the basic shell of things compiling and working in the latest OF and I was but I noticed this add-on had been updated and when I changed some of the function calls to match it and get things comopiling it works but I didn't see messages pass through -- it seemed like "synced" is empty so this is what I needed to do to fix it... it's working ok now, so I think this is a fix that helps -- I can't really follow elsewhere how synced gets set to anything.... |
I found I have the working version used for the SFPC 2020 anniversary exhibition and it has a lot of changes. I dont know why I didn't push it to github. I am testing if it builds and will push it. |
@ofZach That is the code I used for SFPC's 2020 show. |
I don't know if this is an issue or not, but in getting re-coded working on the latest OF, I found I needed to add the following to ofxParameterMidiSync::setSyncGroup
otherwise I didn't see any messages come through to the app via void ofxParameterMidiSync::newMidiMessage (synced.count() always returned zero)
The text was updated successfully, but these errors were encountered: