Skip to content
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

Should we send RPN Null after NRPN? #283

Closed
Zalastax opened this issue Jan 6, 2023 · 4 comments · Fixed by #285
Closed

Should we send RPN Null after NRPN? #283

Zalastax opened this issue Jan 6, 2023 · 4 comments · Fixed by #285

Comments

@Zalastax
Copy link
Contributor

Zalastax commented Jan 6, 2023

Also at MIDI NRPN Null Needed? How? @ Tidal club.

My synth manual and many other manuals contain a statement like this "Once an NRPN parameters
has been specified, all Data Entry messages received on that channel will modify the value of that parameter. To prevent accidents, it is recomended that you set RPN Null (RPN Number = 7FH/7FH) when you have finished setting the value of the desired parameter."

Jwaldmann didn't face issues when not following this approach, so it might not be needed. But I feel it might anyway be best to follow the recommendation.

Should we do as recommended and follow the approach explained here http://www.philrees.co.uk/nrpnq.htm? It would be simple to add here:

if(~nrpn.notNil) {
~val = ~val ? 0;
nrpnLSB = ~nrpn % 128;
nrpnMSB = (~nrpn - nrpnLSB) / 128;
valLSB = ~val % 128;
valMSB = (~val - valLSB) / 128;
schedmidi.value({
midiout.control(chan, 99, nrpnMSB);
midiout.control(chan, 98, nrpnLSB);
midiout.control(chan, 6, valMSB);
midiout.control(chan, 38, valLSB);
});
};

@yaxu
Copy link
Collaborator

yaxu commented Jan 6, 2023

I don't know too much about NRPN, and haven't used it much, but yes it sounds like that's worth doing.

@telephon
Copy link
Contributor

telephon commented Jan 7, 2023

yes, sounds reasonable. @Zalastax do you want to make a pull request?

@Zalastax
Copy link
Contributor Author

Zalastax commented Jan 7, 2023

Yes, I will make a pull request @telephon. I expect to be able to do it in a few weeks time.

@telephon
Copy link
Contributor

telephon commented Jan 7, 2023

thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants