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

add a cross_entropy_over_beam layer. #3639

Merged
merged 15 commits into from
Aug 31, 2017

Conversation

lcy-seso
Copy link
Contributor

@lcy-seso lcy-seso commented Aug 23, 2017

  1. fixes Add a cross entropy beam layer for learning to search model. #3638;
    2. currently this PR cannot pass the unittest before this PR Add a sequence slice layer #3367 is merged.
    * Because this layer cannot work alone, it always works with sequence_slice_layer.
    * To provide a meaningful unittest, sequnence_slice_layer is used, currently this PR cannot pass the unttest.

@lcy-seso lcy-seso requested review from luotao1 and guoshengCS August 23, 2017 10:17
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch 3 times, most recently from db52453 to cdb12dc Compare August 23, 2017 11:30
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch from cdb12dc to 5e59ca7 Compare August 23, 2017 11:31
@lcy-seso lcy-seso changed the title add a cross_entropy_over_beam add a cross_entropy_over_beam layer. Aug 23, 2017
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch 2 times, most recently from 4c1ad17 to 5b96f41 Compare August 25, 2017 02:22
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch from 5b96f41 to 7a42c92 Compare August 25, 2017 04:57
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch from 36976b3 to 7416a18 Compare August 29, 2017 05:46
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch from 7416a18 to 09e903e Compare August 31, 2017 02:25
guoshengCS
guoshengCS previously approved these changes Aug 31, 2017
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch 2 times, most recently from 50c54e9 to 603cf58 Compare August 31, 2017 05:23
@lcy-seso lcy-seso force-pushed the cross_entropy_over_beam branch from 603cf58 to 36f0aa7 Compare August 31, 2017 06:13
@lcy-seso lcy-seso merged commit b66af30 into PaddlePaddle:develop Aug 31, 2017
@lcy-seso lcy-seso deleted the cross_entropy_over_beam branch September 2, 2017 02:00
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.

Add a cross entropy beam layer for learning to search model.
2 participants