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

PyTorch Realtime Inference Music Application Toolkit #2

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

Conversation

setezazi
Copy link
Collaborator

A Max interface for writing up to 3 bars of music and sending back a harmonized version in real time.
Designed for BachNet, suitable for every PyTorch Inference Music Application

A Max interface for writing up to 3 bars of music and sending back a harmonized version in real time.
Designed for BachNet, suitable for every PyTorch Inference Music Application
@sonovice
Copy link
Member

sonovice commented Sep 24, 2020

Thank you very much for this! Would it be enough if we add the Max4BachNet.maxpat to the 3.0 twopass_small branch or did you have to alter the python scripts for this to work?

EDIT: This repo really needs some cleanup...

@setezazi
Copy link
Collaborator Author

setezazi commented Sep 24, 2020

Max4BachNet.maxpat, exsecute.sh and the /XML folder are needed in order for the patch to work. Checkpoints need to be in a /checkpoints directory

EDIT: In fact, inference.py needed to be altered to accept the correctly named .xml file as well as spit out the correctly named harmony.xml in the right folder.
Also, BeamSearch was limited to 3 for each to optimize realtime performance.

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