New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

fix for backwardConnect with latest LSTM from rnn #73

Merged

macournoyer merged 1 commit into macournoyer:master from lerouxrgd:fix/lstm-backward-connect

Feb 1, 2017

lerouxrgd commented Jan 30, 2017

As you can see here:
Element-Research/rnn@b756aeb#diff-252114aaf00f46d74c886c6e9d12c187L40
Logic for backwardConnect has changed.


          fix for backwardConnect with latest LSTM from rnn

acd9c13

macournoyer merged commit 52b648c into macournoyer:master

Owner

macournoyer commented Feb 1, 2017

Thanks!

lerouxrgd deleted the fix/lstm-backward-connect branch

February 1, 2017 05:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment