Add English-Spanish translation problem by voluntadpear · Pull Request #1626 · tensorflow/tensor2tensor · GitHub
Skip to content
This repository was archived by the owner on Jul 7, 2023. It is now read-only.

Add English-Spanish translation problem#1626

Merged
lukaszkaiser merged 1 commit into
tensorflow:masterfrom
voluntadpear:feature/enes
Jul 18, 2019
Merged

Add English-Spanish translation problem#1626
lukaszkaiser merged 1 commit into
tensorflow:masterfrom
voluntadpear:feature/enes

Conversation

@voluntadpear

Copy link
Copy Markdown
Contributor

Hi!

I was able to successfully add and use a English-Spanish translation problem, using Common Crawl, EuroParl v7, UN Corpus and ParaCrawl as my datasets.

I trained both an EN-ES and an ES-EN Transformer models (using the transformer_big_single_gpu hyperparameters set) with this new problem specification.

@googlebot

Copy link
Copy Markdown

@googlebot googlebot added the cla: no PR author has not signed CLA label Jul 8, 2019
@voluntadpear

Copy link
Copy Markdown
Contributor Author

Done CLI

@googlebot

Copy link
Copy Markdown

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes PR author has signed CLA and removed cla: no PR author has not signed CLA labels Jul 8, 2019

@lukaszkaiser lukaszkaiser left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great thanks, looks good!

@lukaszkaiser lukaszkaiser merged commit 69a81e8 into tensorflow:master Jul 18, 2019
@lukaszkaiser

Copy link
Copy Markdown
Contributor

Did you manage to get good results from the models? Great thanks for contributing!

tensorflow-copybara pushed a commit that referenced this pull request Jul 18, 2019
PiperOrigin-RevId: 258838381
@iamyihwa

iamyihwa commented Aug 7, 2019

Copy link
Copy Markdown

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes PR author has signed CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants