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.

Sign up for GitHub

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

Jump to bottom

ENVvStretch #140

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

ENVvStretch #140

wants to merge 3 commits into from

Conversation

friedolino78
Copy link
Contributor

No description provided.

@fundamental
Copy link
Member

The changes all make sense reading through them. Did I miss a corresponding GUI PR such that I can try this out to get a feel for it?

@friedolino78 friedolino78 mentioned this pull request Sep 27, 2021
Open
@fundamental
Copy link
Member

Playing around with something where the note envelope gets longer with increased velocity didn't feel satisfying to me. I'd say for the time being this gets a pass from me, though the general idea of velocity impacting envelopes is an interesting one to explore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fundamental fundamental Awaiting requested review from fundamental

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@friedolino78 @fundamental