mir-dataset-loaders / mirdata

Python library for working with Music Information Retrieval datasets
https://mirdata.readthedocs.io/en/stable/
BSD 3-Clause "New" or "Revised" License
365 stars 58 forks source link

Adding the latest version of compmusic_otmm_makam #551

Closed sertansenturk closed 1 year ago

sertansenturk commented 2 years ago

Hello mirdata people!

As we discussed aeons ago, compmusic_otmm_makam has a fix version that was not available when @genisplaja was implementing the original loader.

It was added to Zenodo (link) later, and it would be nice to have the latest version available in mirdata. I believe it is a small task, and I am happy to make the change myself.

To the best of my understanding, all I need to do is to follow the guideline in contributing.html#datasets-with-multiple-versions:

As the acceptance criteria, I could:

Please let me know your thoughts, if I need to show extra care on some bits & bobs, and please do not hesitate to point out any concerns.

P.S: pinging @nkundiushuti, @genisplaja, @rabitt for visibility

magdalenafuentes commented 1 year ago

@genisplaja is this related to the current loader included in mirdata?

genisplaja commented 1 year ago

Yes! We actually got started with discussions with @sertansenturk on how to push this update forward but it is still pending... I can work on that issue when I'm back from the conferences in India. I think the fix is quite easy to apply, so I'll note it down in my TODO list :)