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

Legato note transitions are not always smooth with portamento enabled #363

Open
Diomendius opened this issue May 24, 2022 · 1 comment
Open

Comments

@Diomendius
Copy link

I believe this is the same issue I discovered and fixed in Yoshimi:
Yoshimi/yoshimi#51
Yoshimi/yoshimi#86

In legato mode with portamento enabled, the first legato transition for a played note will be smooth, but subsequent transitions do not correctly match the phase or other time-dependent parameters from one side of the transition to the other. This causes distortion such as this example, which was created using a simple sine wave ADSynth preset in legato mode with a long portamento duration:
zynaddsubfx_portamento_bad_transition

As I recall, this was caused by the legato code creating a new note but failing to copy over all the necessary fields from the old note. There was also some code to "catch-up" a silent note before unmuting it, which was implemented incorrectly and may still exist in the zyn codebase.

Hopefully the original issue and PR from Yoshimi are still useful, though there is quite a lot to wade through and the codebases have diverged somewhat.

@fundamental
Copy link
Member

Thanks for raising the issue. PRs would be welcome to the zyn codebase.

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

2 participants