nengo / keras-lmu

Keras implementation of Legendre Memory Units
https://www.nengo.ai/keras-lmu/
Other
207 stars 35 forks source link

phased_lstm_keras problem? add_weight () got multiple values for argument 'name' #10

Closed jabowery closed 4 years ago

jabowery commented 4 years ago

See https://github.com/fferroni/PhasedLSTM-Keras/issues/19

arvoelke commented 4 years ago

At first glance this looks like some sort of versioning issue in phased_lstm_keras, which is a separate package. Could you please report the issue there (https://github.com/fferroni/PhasedLSTM-Keras/issues) and let them know what version of keras you're using? If you link it here I can keep an eye on it in case I need to add some sort of version limit to the requirements. I don't think this will affect the LMUCell. Thanks!

jabowery commented 4 years ago

Moved and linked as requested.

arvoelke commented 4 years ago

Closing as this has been moved to the paper branch which is unlikely to change, and this example was removed in #18, which is released on pypi (pip install lmu==0.1.0; https://pypi.org/project/lmu/).