search

RobRomijnders / LSTM_tsc

An LSTM for time-series classification
MIT License
409 stars 146 forks source link

tf.nn.rnn.LSTMcell() #5

Open grwhumphries opened 7 years ago

grwhumphries commented 7 years ago

Hi there,

So it looks like in the latest version of TF (1.0?) - tf.nn.rnn.LSTMcell() has been replaced with tf.contrib.rnn.LSTMcell() - I just upgraded TF today and came across this,

https://github.com/tensorflow/tensorflow/commit/b8101a2b0a7ff1b96999c8e7ad213d1f36b4a4ab

grwhumphries commented 7 years ago

Continuing on that thought.. seq2seq has been moved to tf.contrib.seq2seq.dynamic_rnn_decoder()

SerialDev commented 7 years ago

This actually gives me a:

TypeError: Expected binary or unicode string, got <tensorflow.contrib.rnn.python.ops.core_rnn_cell_impl.MultiRNNCell object

instead as temporary workaround you can still use: tf.contrib.legacy_seq2seq.rnn_decoder()

grwhumphries commented 7 years ago

Thanks - sorry I mean to update this, but you're right, it's legacy_seq2seq