Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

update beam search API in machine translation book example #559

Merged

Conversation

nickyfantasy
Copy link
Contributor

Copy link
Contributor

@jetfuel jetfuel left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@ktlichkid ktlichkid left a comment

Choose a reason for hiding this comment

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

LGTM. Nice Job!

@nickyfantasy nickyfantasy merged commit ef10dd9 into PaddlePaddle:high-level-api-branch Jun 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants