f90 / Wave-U-Net

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

FutureWarning: Using a non-tuple sequence for multidimensional indexing is deprecated #11

Closed loretoparisi closed 5 years ago

loretoparisi commented 5 years ago

It's just a deprecation warning, but it should be use to fix it

/Library/Python/2.7/site-packages/scipy/signal/signaltools.py:2383: FutureWarning: Using a non-tuple sequence for multidimensional indexing is deprecated; use `arr[tuple(seq)]` instead of `arr[seq]`. In the future this will be interpreted as an array index, `arr[np.array(seq)]`, which will result either in an error or a different result.
f90 commented 5 years ago

This is an issue within the scipy library, so I have pretty much nothing to do with this. It seems other people have similar issues, an there is a problem in general with scipy versions below 1.2, see here

I noticed that we are using scipy 1.1 and so probably this will be fixed by updating to 1.2, however it is not available via pip as far as I can tell, so I think we are going to have to ignore it for now.

I am going to leave the issue open so that when scipy 1.2 comes out I will remember to try to update the project to that.

Thanks for reporting.

f90 commented 5 years ago

Scipy 1.2 was released via pip now. If you take the Wave-U-Net virtualenv and upgrade your scipy package via

pip install --upgrade scipy

to install the new 1.2 version, can you verify that this warning disappears?

loretoparisi commented 5 years ago

@f90 okay we will have a look cc!

f90 commented 5 years ago

Closing this since I didn't have this issue anymore after updating...

heavyrick commented 4 years ago

I had the same issue, doing this solved for me:

pip install --upgrade scipy