search

urinieto / msaf

Music Structure Analysis Framework
MIT License
478 stars 79 forks source link

Set mandatory keyword arguments #121

Closed carlthome closed 1 year ago

carlthome commented 1 year ago

What?

Update librosa.frames_to_time call to use keyword arguments.

Why?

MSAF doesn't work with librosa anymore as of https://github.com/librosa/librosa/pull/1431/files

TypeError: frames_to_time() takes 1 positional argument but 3 were given
urinieto commented 1 year ago

Thanks for the PR. Looks good to me!