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

Virtual keyboard: octave shift with note off #5620

Closed
j5v opened this issue Dec 11, 2021 · 3 comments
Closed

Virtual keyboard: octave shift with note off #5620

j5v opened this issue Dec 11, 2021 · 3 comments
Labels
Bug Report Item submitted using the Bug Report template

Comments

@j5v
Copy link
Contributor

j5v commented Dec 11, 2021

Bug Description:
Note-off is sent for the wrong MIDI note when octave shift has been used since note-on.
Likely a bug in JUCE rather than Surge.

Reproduction Steps:

  1. Enable the virtual keyboard in an open Surge window.
  2. Press and hold a key (say "F")
  3. Press C to shift octave
  4. Release the note key ("F")
    The note remains on.

Expected Behavior:
The note ends.

Screenshots:
.

Computer Information (please complete the following information):
Version: Surge XT 0.99.nightly.78fd32b
Build: 2021-12-04 @ 21:39:10 on 'fv-az33-805/pipeline' with 'MSVC-19.29.30137.0' using JUCE 6.1.2
System: Windows 64-bit VST3 on Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Host: Cakewalk by Bandlab @ 44.1 kHz

Additional Information:
.

@j5v j5v added the Bug Report Item submitted using the Bug Report template label Dec 11, 2021
@mkruselj
Copy link
Collaborator

This is #5480 in disguise. All the same reasons.

@j5v
Copy link
Contributor Author

j5v commented Dec 12, 2021

Let's use this as a test case when #5480 is done. If it's because the note-on ID differs from the note-off ID (because the octave transpose happened in the time in between), then the bug will remain.

@mkruselj
Copy link
Collaborator

From what Paul says, #5480 is really really difficult and might not be fixable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Report Item submitted using the Bug Report template
Projects
None yet
Development

No branches or pull requests

2 participants