urinieto / msaf

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

Frames to time takes 1 positional argument but 3 were given #122

Closed nathNath closed 1 year ago

nathNath commented 1 year ago

Hey,

I just took msaf right out of the box, read some docs and some issues up here, but I did not manage to figure out how to overcome this problem. I faced it while trying to execute the quickstart example which is available in the docs https://msaf.readthedocs.io/en/latest/tutorial.html#quickstart

"..\Python\Python310\site-packages\msaf\base.py", line 353, in _compute_framesync_times
    self._framesync_times = librosa.core.frames_to_time(
TypeError: frames_to_time() takes 1 positional argument but 3 were given"

@urinieto can you help me out here with some guidance?

nathNath commented 1 year ago

Just found out that someone figured out how to solve this problem here: https://github.com/urinieto/msaf/pull/121/files I changed my base.py and it works now!

urinieto commented 1 year ago

Glad you figured it out! :)