CRNN-with-STN icon indicating copy to clipboard operation
CRNN-with-STN copied to clipboard

Bi-LTSM's implementation

Open tuanphan09 opened this issue 5 years ago • 1 comments

Your bidirectional LSTM's implementation has a mistake. go_backwards=True in LSTM (rnn_1b, rnn_2b) is only reverse the input before using LSTM, so you have to reverse the output of LSTM before adding or concating with forward-LSTM (rnn_1 , rnn_2).
More info about the issue in here: https://github.com/qjadud1994/CRNN-Keras/issues/26
You can fix it or optimize your code using Bidirectional like this one: https://github.com/tuanphan09/captcha-recognition/blob/master/model.py

tuanphan09 avatar Apr 07 '19 09:04 tuanphan09

Your bidirectional LSTM's implementation has a mistake. go_backwards=True in LSTM (rnn_1b, rnn_2b) is only reverse the input before using LSTM, so you have to reverse the output of LSTM before adding or concating with forward-LSTM (rnn_1 , rnn_2).

More info about the issue in here: qjadud1994/CRNN-Keras#26

You can fix it or optimize your code using Bidirectional like this one: https://github.com/tuanphan09/captcha-recognition/blob/master/model.py

OK thks, I will work on it.

sbillburg avatar May 30 '19 17:05 sbillburg