farizrahman4u / seq2seq

Sequence to Sequence Learning with Keras
GNU General Public License v2.0
3.17k stars 845 forks source link

Error in Importing seq2seq #169

Closed amirveyseh closed 7 years ago

amirveyseh commented 7 years ago

Hi

I've followed your instruction for installing seq2seq and it said seq2seq 0.1.0 was successfully installed. But when I try your to import seq2seq in python I get this error:

Traceback (most recent call last): File "./seq2seqRNN.py", line 1, in import seq2seq File "build/bdist.linux-x86_64/egg/seq2seq/init.py", line 1, in

!/usr/bin/python

File "build/bdist.linux-x86_64/egg/seq2seq/cells.py", line 1, in File "build/bdist.linux-x86_64/egg/recurrentshop/init.py", line 1, in

!/usr/bin/python

File "build/bdist.linux-x86_64/egg/recurrentshop/engine.py", line 3, in ImportError: cannot import name initializations

pradeepbn commented 7 years ago

This requires older version of keras:

pip install https://github.com/fchollet/keras/tarball/1.1.0

DominguesM commented 7 years ago

https://github.com/sunnysai12345/recurrentshop

Best solution at the moment

fritshermans commented 7 years ago

if i use the version above (from sunnysai12345) i get the following error: TypeError: __call__() got an unexpected keyword argument 'name' i'm using python 3.5 with the latest version of keras and tensor flow

0xnurl commented 7 years ago

@fritshermans Line #80 in engine.py should be changed to self.value = init(value) (remove name=name) In any case this fix leads to more errors due to Keras 2 incompatibility.

Blockost commented 7 years ago

Doesn't seq2seq support Keras 2.0 yet ?

farizrahman4u commented 7 years ago

Yes.