Make placeholder time major for RNN.

carlthome / tensorflow-convlstm-cell

A ConvLSTM cell with layer normalization and peepholes for TensorFlow's RNN API.

MIT License

399 stars 150 forks source link

Make placeholder time major for RNN. #14

Closed speedinghzl closed 7 years ago

speedinghzl commented 7 years ago

Hi @carlthome , thanks for your implementation! I noticed that you remove the transposition for inputs in README.md.
This makes me confused. Does the convlstm require inputs with time major?

carlthome commented 7 years ago

TensorFlow 1.2 fixed the transposition inside the RNN API to work with arbitrarily high dimensions, so it's no longer needed.