-
Notifications
You must be signed in to change notification settings - Fork 116
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
Chroma transcription metrics #232
base: main
Are you sure you want to change the base?
Conversation
Thanks Caleb. Apart from PEP8 stuff, you'll also need to add separate evaluation for chroma metrics to Do we have any implementations to test this against? It's a small enough change that I am ok with not testing it against something existing, though we should be very certain in its correctness then :) |
yes
yes, see here. |
The music-ir link is dead for me. Do you know of any others? |
The MIREX website is temporarily down, my understanding is that they're working on it. I'll check if I have the result files stored locally somewhere, otherwise we'll have to wait till it's back up. |
Now that the MIREX website is back up, can we move forward on this? |
Yes, I'll try to wrap it up in the next couple days |
I'm a bit confused about comparing to MIREX results. The existing output*.json files don't match the differences from this comment, so I'm not sure what to look for when comparing the results. |
@justinsalamon can you provide some guidance here? |
Bumping this one, since it seems like everyone disappeared after summer? |
Is it still planned to integrate this functionality and does anybody know which points are still open? |
This needs to be picked up again. I don't think anyone is planning on doing so. |
Implements and tests #197