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

Voice deactivation wrong in unison mode #195

Open
image-et-son opened this issue Sep 8, 2022 · 2 comments
Open

Voice deactivation wrong in unison mode #195

image-et-son opened this issue Sep 8, 2022 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@image-et-son
Copy link
Owner

The deactivation of voices is not properly taken into account in unison mode, i.e. deactivated voices play anyway. Also, when no voice is activated, playing any key in unison crashes the firmware completely.

@image-et-son image-et-son added the bug Something isn't working label Sep 8, 2022
@image-et-son image-et-son self-assigned this Sep 8, 2022
@jmechnich
Copy link

Not sure if this is related (or even normal / expected behaviour?):
In unison mode with priority last, if one plays and holds in series C-D-E-F-G and starts releasing the keys from the top down, the sounding pitches will be:

PRESSED   - SOUNDS
C         - C
C-D       - D
C-D-E     - E
C-D-E-F   - F
C-D-E-F-G - G
C-D-E-F   - C
C-D-E     - D
C-D       - C
C         - C

My naive expectation was that the 'history' of key presses is kept to a certain point but after looking at the code it does not seem to be the case. Anyways, it seems a bit strange that it will keep switching between C and D after each released key (try more than five keys 😄).

@image-et-son
Copy link
Owner Author

Hi! No, that's different issue, but I have already created a ticket for it: #106. But well spotted!

I think it would be proper to take the timestamp into account, but I am not sure if it is worth the effort (or rather: the additional firmware storage space that would cost).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants