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

mir_eval.key.validate(reference_key, estimated_key)¶ when the key or the mode are unknown #333

Open
PRamoneda opened this issue Oct 21, 2020 · 3 comments

Comments

@PRamoneda
Copy link
Contributor

PRamoneda commented Oct 21, 2020

Hello,

First of all, thank you very much for the library. It is fantastic!

At mir_eval.key:

When you don't know the key or the mode (in the reference or in the estimated) you should be able to compute the metric.

see the examples :

x minor
C

Well, x minor can be computed but I think that if your algorithm is performing the mode, the score should be more than 0.

Thank you so much!

@PRamoneda PRamoneda changed the title mir_eval.key.validate(reference_key, estimated_key)¶ when you don't know the key the mode mir_eval.key.validate(reference_key, estimated_key)¶ when the key or the mode are unknown Oct 21, 2020
@PRamoneda
Copy link
Contributor Author

On the other hand, why do you only allow to evaluate (key, modes) that have major or minor modes? I believe that there are key datasets with much more extended vocabularies.

Thank you so much!!

@craffel
Copy link
Owner

craffel commented Oct 21, 2020

The key module was created to match the conventions used by MIREX. If there are other benchmarks that use a richer set of annotations, we welcome PRs to implement that kind of evaluation.

@PRamoneda
Copy link
Contributor Author

Hey @craffel!
angel faraldo
As is seen in this Angel Faraldo phd Thesis figure there are some datasets with extended vocabularies. (I think that all the vocabularies can be reduced to diatonic modes).

image

There are more datasets with diatonic modes annotations. As:
https://zenodo.org/record/1290737#.X5BjPC3FRhE
https://zenodo.org/record/1095691#.X5BjUi3FRhE
https://zenodo.org/record/1101082#.X5BjaC3FRhE

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

No branches or pull requests

2 participants