-
Notifications
You must be signed in to change notification settings - Fork 343
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
Added ADSR envelope with velocity sensitivity #1832
Conversation
referencing #1811 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks generally good, just small nitpicks. Make sure to add your change to the CHANGELOG file as well.
If these changes work for you, is it too much trouble to ask you to do a build with py3.10? I had no trouble getting this working on linux, but this windows stuff is a nightmare |
Never mind about the build, I got it working, something about defaulting to 32bit even when using the x64 developer console, idfk... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now. Thanks!
@fabricatedmath For the future, builds for all platforms for any pull request can be download from its build artifacts page. |
Oh wow good to know, thanks! All the stuff I downloaded and installed to build this on windows feels like malware |
Github won't let me attach a midi file...
But I set the frame rate to 30 fps, attack to 0.1, decay to 0.1, sustain to its default 0.6 and release to 0.1, First note in my file plays for 0.5s, so attack takes 3 frames, decay takes 3 frames and at frame 15, release happens for 3 frames and at frame 18 its back to 0. Seems to work for Single mode and All mode.
Velocity sensitivity seems to be working as well.
Let me know what you think!