csteinmetz1 / auraloss

Collection of audio-focused loss functions in PyTorch
Apache License 2.0
695 stars 66 forks source link

Librosa warning when using MelSTFTLoss #33

Closed csteinmetz1 closed 1 year ago

csteinmetz1 commented 2 years ago

We get the following warning due to changes in the librosa interface. Should be easy to address.

env/lib/python3.7/site-packages/auraloss/freq.py:138: FutureWarning: Pass sr=44100, n_fft=1024 as keyword args. From version 0.10 passing these as positional arguments will result in an error

sivannavis commented 1 year ago

Hi @csteinmetz1 , could you please tell me if you solved this? I tried to add something like librosa.filters.mel(sr=self.sample_rate, n_fft=self.n_fft, n_mels=self.n_mels)), but the warning still shows up. Thank you!

csteinmetz1 commented 1 year ago

This warning should now be fixed in #44 and there is a new release v0.3.0