j-min / tf_tutorial_plus

Tutorials for TensorFlow APIs the official documentation doesn't cover
289 stars 98 forks source link

core_rnn_cell_impl not in ops.core_rnn_cell_impl in TF1.2 #2

Open A-Atef-NLP opened 7 years ago

A-Atef-NLP commented 7 years ago

Thanks A Lot for your tutorial.

To support TF 1.2 I think you need to update below line "from tensorflow.contrib.rnn.python.ops.core_rnn_cell_impl import " To "from tensorflow.contrib.rnn import "

Also to use "embedding_attention_seq2seq" set include path or use: tf.contrib.legacy_seq2seq.embedding_attention_seq2seq

Thanks Again for your tutorial.

j-min commented 7 years ago

My pleasure! Right, as you said some codes are incompatible with TF 1.2, and I plan to update them (+ with new features, when I feel not so lazy :P )