Script is sending the wrong Velocity Value (which is being covered by C12, but may not be by future versions).

Question

Script is sending the wrong Velocity Value (which is being covered by C12, but may not be by future versions).

djaychela opened this issue a year ago · comments

I don't want to break an NDA, but let's just say I know that this will not work in a future version of Cubase (I think that C12 is masking this error, and a later version will not).

I understand Python well, but don't have any typescript skills, so couldn't do a PR, however, I think in this file:

src/midi/PortPair.ts

There is an issue which is a simple fix:

output.sendMidi(context, [0x90, pitch, +Boolean(velocity) * 0xff]);

Should read

output.sendMidi(context, [0x90, pitch, +Boolean(velocity) * 0x7f]);

So that the script tries to send 127 as the maximum value, not 255. Then it will work.

Darren Jones · Answer 1 · Thu Aug 24 2023 01:29:47 GMT+0800 (China Standard Time)

I've made a manual change to the compiled JS file (line 4146 for me) and it works fine, BTW.

bjoluc · Answer 2 · Thu Aug 24 2023 02:31:01 GMT+0800 (China Standard Time)

A message from the future, exciting! Thanks for the heads-up @djaychela, and what a silly mistake 🤦 Fixed by c7d5597 which will be included in 1.7.0 soon. By the way, any words on the 2024 US election yet?

github-actions · Answer 3 · Thu Sep 21 2023 22:53:18 GMT+0800 (China Standard Time)

🎉 This issue has been resolved in version 1.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀