auraloss icon indicating copy to clipboard operation
auraloss copied to clipboard

Can librosa not be a strict dependency?

Open turian opened this issue 1 year ago • 7 comments

Given different versions of OSes with different versions of librosa, librosa is a huge pain to install on certain machines.

For example, I use lambdalabs GPUs and the librosa is wrong. This old librosa depends upon a particular pinned version of numba that's a huge pain to install:

ImportError: Numba needs NumPy 1.22 or less

Thus, I think librosa should be deprecated.

turian avatar Aug 15 '22 22:08 turian

Hey @turian. Thanks for raising this. I think we can add librosa as an optional dependency since its only needed for the mel filterbanks. Is scipy also a problem for you? I think we could move to be optional as well potentially.

csteinmetz1 avatar Aug 16 '22 00:08 csteinmetz1

If you want to take a look, I have started a branch that moves librosa and scipy to optional install by default.

csteinmetz1 avatar Aug 16 '22 00:08 csteinmetz1

scipy is generally fine. Just librosa with it's weird numba pin sucks so much.

turian avatar Aug 16 '22 11:08 turian

I am happy the beta test when the branch is ready!

turian avatar Aug 16 '22 11:08 turian

Great, could you give the librosa branch a try? I did some basic testing and things seemed to be working for me.

csteinmetz1 avatar Aug 16 '22 13:08 csteinmetz1

pip3.8  install --upgrade git+https://github.com/csteinmetz1/auraloss.git@librosa

appears to work. If you submit a PR, I can also code review the diff if you like

turian avatar Aug 20 '22 01:08 turian

Thanks, see #37.

csteinmetz1 avatar Aug 20 '22 01:08 csteinmetz1

Should be solved with #37 now and merged to main.

csteinmetz1 avatar Jan 09 '23 21:01 csteinmetz1