sednn icon indicating copy to clipboard operation
sednn copied to clipboard

Global name 'librosa' is not defined

Open Reagan1947 opened this issue 4 years ago • 5 comments

Sednn/mixture2clean_dnn/preparedata.py

def read_audio(path, target_fs=None):
    (audio, fs) = soundfile.read(path)
    if audio.ndim > 1:
        audio = np.mean(audio, axis=1)
    if target_fs is not None and fs != target_fs:
        audio = librosa.resample(audio, orig_sr=fs, target_sr=target_fs)
        fs = target_fs
    return audio, fs
  • librosa is not define, Should import pack librosa.
  • Moreover librosa will also cause TypeError: expected string or buffer pip install 'joblib==0.11' --force-reinstall will solve it.

Reagan1947 avatar Sep 04 '19 02:09 Reagan1947

pip install librosa

On Tue, 3 Sep 2019 at 19:42, Reagan lee [email protected] wrote:

Sednn/mixture2clean_dnn/preparedata.py

def read_audio(path, target_fs=None): (audio, fs) = soundfile.read(path) if audio.ndim > 1: audio = np.mean(audio, axis=1) if target_fs is not None and fs != target_fs: audio = librosa.resample(audio, orig_sr=fs, target_sr=target_fs) fs = target_fs return audio, fs

  • librosa is not define, Should import pack librosa.
  • Moreover librosa will also cause TypeError: expected string or buffer pip install 'joblib==0.11' --force-reinstall will solve it.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/yongxuUSTC/sednn/issues/47?email_source=notifications&email_token=ABJGHUQ7UNVO5YSQPGOHINDQH4OBRA5CNFSM4ITMWCBKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HJEFRFQ, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJGHUQAZ4KEXNFRADHQQ53QH4OBRANCNFSM4ITMWCBA .

yongxuUSTC avatar Sep 04 '19 06:09 yongxuUSTC

Is import of "librosa" package a must? at preparedata.py

Reagan1947 avatar Sep 07 '19 12:09 Reagan1947

Hi,

You should install from librosa source, it is another library. It is not our code. https://librosa.github.io/librosa/

You can install use "pip install librosa --user"

Best regards, yong

On Mon, 9 Sep 2019 at 23:58, Reagan lee [email protected] wrote:

Yes, you must import librosa. Best wishes, Qiuqiang … <#m_2190366584975570790_> ________________________________ From: Reagan lee [email protected] Sent: 07 September 2019 13:03 To: yongxuUSTC/sednn [email protected] Cc: Subscribed [email protected] Subject: Re: [yongxuUSTC/sednn] Global name 'librosa' is not defined (#47 https://github.com/yongxuUSTC/sednn/issues/47) Is import of "librosa" package a must? at preparedata.py — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FyongxuUSTC%2Fsednn%2Fissues%2F47%3Femail_source%3Dnotifications%26email_token%3DADFXTSLIC5WOMSZO76PX6K3QIOKC3A5CNFSM4ITMWCBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6EXLSY%23issuecomment-529102283&data=02%7C01%7Cq.kong%40surrey.ac.uk%7C8e5485f97cda49a6a8e608d7338b77e9%7C6b902693107440aa9e21d89446a2ebb5%7C0%7C0%7C637034546414116986&sdata=4fSjvcL%2FiMsRyjHvCHZWpSfG5dLLNoctUvZMvRB%2FKkc%3D&reserved=0, or mute the thread https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADFXTSOVWSV3QNX2YSR6QTLQIOKC3ANCNFSM4ITMWCBA&data=02%7C01%7Cq.kong%40surrey.ac.uk%7C8e5485f97cda49a6a8e608d7338b77e9%7C6b902693107440aa9e21d89446a2ebb5%7C0%7C0%7C637034546414116986&sdata=SsPFpcosxFzXwvttK24LddVB1PmATkTy99Nbtkp7IFU%3D&reserved=0 .

But i don't find in 'Sednn/mixture2clean_dnn/preparedata.py'

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/yongxuUSTC/sednn/issues/47?email_source=notifications&email_token=ABJGHUXI5PPNAZIZZWA5ORDQI5APLA5CNFSM4ITMWCBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6KBU5Y#issuecomment-529799799, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJGHUQRMGJUG63OHZ7VYCLQI5APLANCNFSM4ITMWCBA .

yongxuUSTC avatar Sep 10 '19 17:09 yongxuUSTC