urinieto / msaf

Music Structure Analysis Framework
MIT License
490 stars 78 forks source link

fix bug in find_estimation when labels_id is None #111

Closed techhero-es closed 3 years ago

techhero-es commented 3 years ago

Hi,

The Collection Mode example in the Run MSAF notebook throws an exception because the first parameter to re.match() isn't a string or compiled regex. I am using Python 3.8.2 and fixed it by expanding the existing check in find_estimation. Not sure if this is the most elegant solution though.

Best regards, Matthias

urinieto commented 3 years ago

Looks good to me, thanks!