We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
From the code and Pascanu et. al, I think that Eqn 7 in the arXiv version (or Eqn 2 in the ICML version) should add a tanh after L_0.
tanh
L_0
And is a y_t missing in the same equation as well? From Bahdanau et al. Appendix part Page 14, p(y_i | s_i, y_{i-1}, c_i)
y_t
p(y_i | s_i, y_{i-1}, c_i)
The text was updated successfully, but these errors were encountered:
We'll push an updated version of the paper to arxiv soon and clarify this. Thanks for pointing it out!
Sorry, something went wrong.
No branches or pull requests
From the code and Pascanu et. al, I think that Eqn 7 in the arXiv version (or Eqn 2 in the ICML version) should add a
tanh
afterL_0
.And is a
y_t
missing in the same equation as well? From Bahdanau et al. Appendix part Page 14,p(y_i | s_i, y_{i-1}, c_i)
The text was updated successfully, but these errors were encountered: