-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[Dy2stat] Add Ptb Lm Test for V2 APIs #27461
[Dy2stat] Add Ptb Lm Test for V2 APIs #27461
Conversation
… refine_api2, test=develop
… refine_api2, test=develop
Thanks for your contribution! |
… refine_api2 test=develop
…nto refine_api_test_ptb_lm, test=develop
… refine_api_test_ptb_lm, test=develop
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Mark paddle.fluid.layers.reshape -> paddle.reshape
Not only that, I found more and I'm going to change |
Refine all fluid APIs except Embedding which is different to 2.0 Embedding |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR types
Others
PR changes
Others
Describe
Add Ptb lm Test for V2 APIs.
We used tool from https://www.paddlepaddle.org.cn/documentation/docs/zh/2.0-beta/guides/migration_cn.html#paddle1-xpaddle2-0beta to do v1.x to v2 transformation.
This PR pulled changes from #27430, please DO NOT merge before #27430 is merged