Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bi-LTSM's implementation #12

Open
tuanphan09 opened this issue Apr 7, 2019 · 1 comment
Open

Bi-LTSM's implementation #12

tuanphan09 opened this issue Apr 7, 2019 · 1 comment

Comments

@tuanphan09
Copy link

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

@sbillburg
Copy link
Owner

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants