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

Dragging up/down vertical piano results in stuck notes. #5028

Closed
programnoir opened this issue Jun 9, 2019 · 9 comments
Closed

Dragging up/down vertical piano results in stuck notes. #5028

programnoir opened this issue Jun 9, 2019 · 9 comments
Assignees

Comments

@programnoir
Copy link

image

Steps to reproduce:

  1. Create a piano pattern on the song editor or beat/bassline editor.
  2. Drag mouse down the piano keys.

Expected behavior: Each key presses and releases as I go up and down the piano.

Actual result: Multiple piano keys graphically remain pressed (the keys do not audibly stick)

@ubailey1
Copy link

I can confirm this problem as well. It seems to be random, like every other loop, some keys dont' play sound

@musikBear
Copy link

Not in win32 1.2.0 rc8 (tested in both SE ans B&E contained PRoll)
Is this with a specific VST?
Please retry with default presets

@DomClark
Copy link
Member

Reproduced.

@musikBear This is in 1.2.0, not 1.2.0-rc8. Please update to the latest version.

@ubailey1
Copy link

I tested in version 1.2

@BaraMGB
Copy link
Contributor

BaraMGB commented Jun 27, 2019

It's this commit: 461facc
@DomClark can you have a look at this?

@DomClark
Copy link
Member

DomClark commented Jul 4, 2019

I think it's a threading issue - previously all MIDI events relating to mouse input were sent from the UI thread, but since that commit some come from the processing thread too. I guess if one drags too quickly, the note-on arrives at the piano model after the note-off. Strangely, this doesn't seem to affect all instruments - single-streamed ones are fine, along with Mallets, Organic, sfxr, and Vibed. I'll investigate.

@DomClark DomClark self-assigned this Jul 4, 2019
@enp2s0
Copy link
Contributor

enp2s0 commented Jul 11, 2019

I also notice this issue, especially with zasfx. This occurs under both low and high CPU load, so it doesn't seem to be affected by it. Especially with my FadeButton patch, this bug is particularly annoying as it results in FadeButtons getting stuck in their held-note color even when no note is playing.

@karmux
Copy link
Contributor

karmux commented Sep 8, 2019

Related to #3005.

@zonkmachine
Copy link
Member

Closed in #5127

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

No branches or pull requests

8 participants