f90 / Wave-U-Net

Implementation of the Wave-U-Net for audio source separation
MIT License
845 stars 177 forks source link

Requirements for CPU on macOS #10

Closed loretoparisi closed 5 years ago

loretoparisi commented 5 years ago

These are the right requirements when on macOS (hence no NVIDIA, no GPU :(

sacred==0.7.3
#tensorflow-gpu==1.8.0
tensorflow == 1.8
librosa==0.6.2
scikit-image==0.13.1
soundfile==0.10.2
scikits.audiolab==0.11.0
lxml==4.2.1
musdb==0.2.3
museval==0.2.0
google
protobuf

Note that both google and protobuf are needed - according to https://github.com/tensorflow/tensorflow/issues/6341

Thank you.

f90 commented 5 years ago

I will add a notice that for non-GPU setups, you need to replace the TF version like you did.

Can you name me the google and protobuf version you end up with when doing this? Provided it works now? I would like to fix the versions of the packages to avoid conflicts in the future when they are updated

f90 commented 5 years ago

FYI I was able to make it work right out of the gate on a Linux system with libsndfile pre-installed, when using this requirements file with pip, in a completely new pip virtualenv (no other packages):

numpy==1.15.4
sacred==0.7.3
tensorflow==1.8.0
librosa==0.6.2
scikit-image==0.13.1
soundfile==0.10.2
scikits.audiolab==0.11.0
lxml==4.2.1
musdb==0.2.3
museval==0.2.0
google==2.0.1
protobuf==3.4.0

Could you verify this is okay to use on your end as well? Thanks

loretoparisi commented 5 years ago

Let me try

loretoparisi commented 5 years ago

@f90 confirmed. Now I get the errors that I have in https://github.com/f90/Wave-U-Net/issues/9, so it's ok with this requirements.