MTG / DeepConvSep

Deep Convolutional Neural Networks for Musical Source Separation
GNU Affero General Public License v3.0
469 stars 110 forks source link

Theano 0.9.0 API change #7

Closed ronggong closed 7 years ago

ronggong commented 7 years ago

Traceback (most recent call last): File "/homedtic/rgong/DeepConvSep/trainCNN.py", line 42, in import lasagne File "/homedtic/rgong/keras_env/lib/python2.7/site-packages/lasagne/init.py", line 24, in from . import layers File "/homedtic/rgong/keras_env/lib/python2.7/site-packages/lasagne/layers/init.py", line 7, in from .pool import * File "/homedtic/rgong/keras_env/lib/python2.7/site-packages/lasagne/layers/pool.py", line 6, in from theano.tensor.signal import downsample ImportError: cannot import name downsample

Theano has changed its API for the 0.9.0 version. lasagne haven't updated yet.

max_pool_2D method doesn't exist anymore in ``downsample".

I found the solution for this issue in https://github.com/Theano/Theano/issues/4337

ghost commented 7 years ago

I use Theano-0.9.0rc1 with Lasagne-0.2.dev1.

And I don't get that error. Do you have those versions installed?

ronggong commented 7 years ago

Theano on my machine is 0.9.0rc2

The documentation indicates this API change:

http://deeplearning.net/software/theano_versions/dev/library/tensor/signal/pool.html?highlight=pool_2d#theano.tensor.signal.pool.pool_2d

nkundiushuti commented 7 years ago

I removed the maxpool layer in the training. however, we need to re-train the model