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

Keep pedaling (sustain or sostenuto) effective when the same note is played more than once. #905

Merged
merged 2 commits into from
Jun 11, 2021

Conversation

jjceresa
Copy link
Collaborator

@jjceresa jjceresa commented Jun 4, 2021

When playing the same note more than once the previous note is forced in release stage except if pedaling (sostenuto or sustain) is active. This makes it sound more like a real piano. See https://lists.nongnu.org/archive/html/fluid-dev/2021-06/msg00001.html

jjceresa added 2 commits June 4, 2021 19:48
previous note is forced in release stage except if
pedaling (sostenuto or sustain) is active.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 4, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@jjceresa
Copy link
Collaborator Author

jjceresa commented Jun 4, 2021

@mrbumpy409 This PR should fix the behaviour you started on the mailing list (https://lists.nongnu.org/archive/html/fluid-dev/2021-06/msg00001.html).
Please, could you have a try on your side to confirm that it works as expected ?.

Copy link
Member

@derselbst derselbst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks correct to me. But I would like to test this behaviour with overlapping notes in the next days. Can you remember why you wanted pedaling to be ignored back then?

@jjceresa
Copy link
Collaborator Author

jjceresa commented Jun 5, 2021

Can you remember why you wanted pedaling to be ignored back then?

If I remember well, I noticed that when playing the same note more than once while pedaling this result in sound growing louder. I was thinking that this wasn't natural. I was wrong because in fact this is very natural on real acoustic piano for which keeping the previous note still alive allows a fluid legato playing with following notes played at different dynamics (i.e velocity).

@jjceresa
Copy link
Collaborator Author

jjceresa commented Jun 5, 2021

But I would like to test this behaviour with overlapping notes in the next days.

For overlapping same note, new voices (with respective velocity) will be started for each following noteon. On noteoff these voices will be sustained by the active pedal. Then when the pedal become inactive, all voices are released at the same time.

Copy link
Member

@derselbst derselbst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well, thanks!

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 this pull request may close these issues.

2 participants