-
Notifications
You must be signed in to change notification settings - Fork 242
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
Motivation for x_context #1
Comments
Yes in the paper it is not mentioned anywhere but the code has it. https://github.com/abisee/pointer-generator/blob/master/attention_decoder.py#L150 |
I get the paper where similar kind of attention mechanism is used. |
Thanks a lot for this work! |
Thanks for pointing this out. You are right. I have updated my code, I still need to re-run the experiments though. I will update the result after that. Here is the commit: |
I am curious about the motivation for this step, as I couldn't anything about it in See et al., though I might have just missed it.
The text was updated successfully, but these errors were encountered: