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

Seq2seq tutorial #108

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Seq2seq tutorial #108

wants to merge 3 commits into from

Conversation

jumerckx
Copy link

As suggested by @MikeInnes in #69, I'm making a pull request with a seq2seq tutorial in literate.jl format.

I'm hoping someone could have a look at this model since performance (translation quality) isn't very good.

Thanks

@MikeInnes
Copy link
Member

Would be nice to figure out the training issues so that this can get more interesting results before we merge. Also, you shouldn't need to commit the ipynb file here since we have a conversion script that will generate it from the latest .jl version.

@jumerckx
Copy link
Author

Thanks for the feedback, I've removed the notebook version.
As for the translation quality, I'm afraid I'm unable to fix this, although I'm pretty certain I'm making a silly mistake somewhere.

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

Successfully merging this pull request may close these issues.

2 participants