-
Notifications
You must be signed in to change notification settings - Fork 377
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
Copy attention based decoder #153
Comments
@AtmaHou we'll see if it can be integrated in the next release. You're welcome to submit a PR. We need all the help we can get. |
Thanks for your reply, it will be in great need. |
Any Update on this? |
@varunAI Yup working on it. Will push to develop as soon as it's done. |
@pskrunner14 I can work on this if you need some help. |
@pskrunner14 Any news on this ? |
@pskrunner14 is this PR #172 reliable for the copy mechanism ? |
@Diego999 no, I'm facing some issues with dimensions as mentioned and there seems to be a problem with the dynamic dictionary too. I'll get to this asap. Will post any updates on this thread. |
@pskrunner14 Thank you for your quick answer ! |
@pskrunner14 Do you have any news about it ? Thanks |
You guys really did an excellent work with high scalability. Better than OpenNMT in my opinion :)
Copy mechanism could greatly enhance the performance of Seq2Seq model, would you guys implement a decoder with copy mechanism?
The text was updated successfully, but these errors were encountered: