Skip to content

Comments

Adding the distributed implementation for machine translation#7751

Merged
kavyasrinet merged 6 commits intoPaddlePaddle:developfrom
kavyasrinet:machine_translation
Jan 23, 2018
Merged

Adding the distributed implementation for machine translation#7751
kavyasrinet merged 6 commits intoPaddlePaddle:developfrom
kavyasrinet:machine_translation

Conversation

@kavyasrinet
Copy link

@kavyasrinet kavyasrinet commented Jan 22, 2018

Fixes issue: #7745
Testing the implementation right now.

@kavyasrinet
Copy link
Author

Tested the implementation on my machine and it works.

Copy link
Contributor

@helinwang helinwang left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@kavyasrinet kavyasrinet merged commit 5466eff into PaddlePaddle:develop Jan 23, 2018
@kavyasrinet kavyasrinet deleted the machine_translation branch January 24, 2018 01:04
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