Well I can’t  see anything wrong either, sorry. But one thing I suggest u to do is that both  the cell state and hidden state are zeroes by default, so no need to set them manually. Also, u shouldn’t be using Variable anymore and that the states should be  tensors.