forked from interactiveaudiolab/penn
-
Notifications
You must be signed in to change notification settings - Fork 0
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
New network design: 60 midi notes + offsets #9
Comments
Well, the new config is almost ready for training, there are some problems left for tomorrow, like there's still no metrics or the offset is calculated on a cpu instead of the gpu. Fix it! |
anthonio9
added a commit
that referenced
this issue
Feb 20, 2024
anthonio9
added a commit
that referenced
this issue
Feb 20, 2024
Take a look here to resolve the problem with images not being logged with wandb |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This issue is about the introduction of a new network model, where the logits layer instead of 61440 pitch bins has 660*2 midi bins. Each of the 60 values in the first column corresponds to a separate midi note. The second column tells how big the pitch deviation is from that midi note, 0 being -50, 59 being +50 cents.
Conversion to this new format is done with
penn.data.preprocess.core.note_dict_to_pitch_dict60()
where 60 obviously stands for the number of midi notes.The text was updated successfully, but these errors were encountered: