about concat about convolutional_lstm_pytorch HOT 5 CLOSED

@automan000 Guess you have referenced some implementations of LSTM instead of convLSTM, note the difference of formulas of them .
LSTM

convLSTM

The raw LSTM really doesn't involve c for computing the gate while convLSTM does.

from convolutional_lstm_pytorch.

Hi Chen,
I think you might be right. I forget why I did this when I wrote the code.
I have checked several implementations. Many of them make the same decision.
I will do some further investigations. Once finished, I will let you know.
Thanks anyway.

from convolutional_lstm_pytorch.

I have some of my own opinions, which may be to save code complexity and the number of parameters

from convolutional_lstm_pytorch.

@Jackie-Chou Actually I am referring the implementations such as BasicConvLSTMCell
But your words are very convincing. I will update the code ASAP. Thanks a lot.

from convolutional_lstm_pytorch.

Bug fixed. Close this issue.

from convolutional_lstm_pytorch.