You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I found an issue in the code dnsmos_local.py where the librosa.resample() function is called without specifying the required keyword-only arguments orig_sr and target_sr.
This change ensures that the original sampling rate (input_fs) and the target sampling rate (fs) are correctly specified when resampling the audio data. This modification resolves an error where the resample() function is called with missing required keyword-only arguments.
Please consider updating the code accordingly. Thank you!
The text was updated successfully, but these errors were encountered:
so the arguments shouldn't cause a positional argument error. I do agree that it would be better if it were changed though. They could use newer version of librosa in future revisions.
I found an issue in the code
dnsmos_local.py
where thelibrosa.resample()
function is called without specifying the required keyword-only argumentsorig_sr
andtarget_sr
.The current code:
audio = librosa.resample(aud, input_fs, fs)
Should be updated to:
audio = librosa.resample(aud, orig_sr=input_fs, target_sr=fs)
This change ensures that the original sampling rate (
input_fs
) and the target sampling rate (fs
) are correctly specified when resampling the audio data. This modification resolves an error where theresample()
function is called with missing required keyword-only arguments.Please consider updating the code accordingly. Thank you!
The text was updated successfully, but these errors were encountered: