Skip to content
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.

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

Encoder and Decoder RNN #2

Open
ShishirPatil opened this issue May 14, 2019 · 1 comment
Open

Encoder and Decoder RNN #2

ShishirPatil opened this issue May 14, 2019 · 1 comment

Comments

@ShishirPatil
Copy link

The paper - Semi-supervised Sequence Learning(https://arxiv.org/abs/1511.01432) - states that for training of SA-LSTM, they used the same LSTM for both encoding and for decoding. However, this implementation uses 2 LSTMs - encoder_cell, and decoder_cell. Could you please clarify.

Thanks,
Shishir

@miguelmartin75
Copy link

miguelmartin75 commented May 21, 2019

Likely was just a mistake on the author's (github author) part. Seems to work regardless, just requires more parameters.

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

No branches or pull requests

2 participants