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

experiments/piano_recital: piano voice test case and demo #283

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

Conversation

matt-har-vey
Copy link

This resulted from a piano player (yes, me) "tracking" to MIDI files and then trying to get the best sound from AMY seen as a batch-renderer.

The Chopin ones double as a stress test for voice stealing under heavy sustain and variable velocity.

It might also be a good starting place for making voice stealing steal the quietest note instead of the oldest.

Also note there's a bug fix hiding in it. I found amy.send(volume=value). I guess it would properly be tracked by a separate issue, but it's a one-character change...

Lowercase v is velocity. Capital V is volume.
This resulted from a piano player "tracking" to MIDI files and then
trying to get the best sound from AMY seen as a batch-renderer.

The Chopin ones double as a stress test for voice stealing under
heavy sustain and variable velocity.
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.

1 participant