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

Clean text.py and decode.py for API 2.0【cherry-pick 26853】 #27958

Conversation

LiuChiachi
Copy link
Contributor

@LiuChiachi LiuChiachi commented Oct 15, 2020

PR types

New features

PR changes

APIs

Describe

  1. 删除text.py
  2. 整理解码相关API,增加paddle.fluid.layers.rnn下dynamic_decode的动态图支持,并将其与BeamSearchDecoder导入paddle.nn.decode下,将gather_tree导入paddle.nn.functional下

image

#26853

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@LiuChiachi LiuChiachi force-pushed the add-dy-decode-cherry-pick-2.0rc branch from 1f17999 to 6b85421 Compare October 16, 2020 05:55
* Make dynamic_decode support dygraph and expose to API 2.0
test=develop

* update info about BeamSearchDecoder and dynamic_decode

* remove all APIs in paddle.text, expose BeamSearchDecoder and dynamic_decode

* update example code

* delete test_text.py, decode.py, update some doc, fix example code float64

* delete decode import from paddle.nn

* fix unittest bugs

* use dygraph.Embedding instead of nn.Embedding, add paddle.enbale_static()

* update, correct doc

* move dynamic_decode, BeamSearchDecoder API to paddle.nn

* fix code style

* update unittest param, delete import pf text.py

* set dtype of beamsearchtest float64

* update example code of BeamSearchDecoder, dynamic_decode

Co-authored-by: LiuChiaChi <[email protected]>
@LiuChiachi LiuChiachi force-pushed the add-dy-decode-cherry-pick-2.0rc branch from 6b85421 to 4d93359 Compare October 16, 2020 06:05
@LiuChiachi LiuChiachi changed the base branch from develop to release/2.0-rc October 16, 2020 06:06
@LiuChiachi LiuChiachi closed this Oct 16, 2020
@LiuChiachi LiuChiachi reopened this Oct 16, 2020
@LiuChiachi LiuChiachi closed this Oct 16, 2020
@LiuChiachi LiuChiachi reopened this Oct 16, 2020
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Oct 16, 2020
@PaddlePaddle PaddlePaddle unlocked this conversation Oct 16, 2020
Copy link

@saxon-zh saxon-zh 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
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LGTM

@guoshengCS guoshengCS merged commit 4316bd4 into PaddlePaddle:release/2.0-rc Oct 17, 2020
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.

5 participants