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

Update 2.0 Save/Load API names/arguments/doc examples #27138

Conversation

chenwhql
Copy link
Contributor

@chenwhql chenwhql commented Sep 7, 2020

PR types

Function optimization

PR changes

APIs

Describe

关于save load的一些API形态改动,为避免2.0-beta宣传后,2.0-RC API又发生变动,这里cherry-pick一下,相关功能之前已经在2.0-beta中,本次cherry-pick主要改动接口形态

Cherry-pick PRs:

* update set_dict method name & add aliases

* fix var name error

* fix alias formats

* use set_state_dict in unittest

* add decorator solve compatible problem

* polish decorator

* replace layer set_state_dict by patched method

* remove import monkey path layer

* fix import function error

* add unittest for coverage
…ddlePaddle#26718)

* support load infer model format state dict

* add unittests

* remove keep name table

* recolve circle inport

* fix compatible problem

* recover unittest

* polish doc and comment
…Paddle#27056)

* change configs to config & update examples

* fix deprecate decorator conflict
@paddle-bot-old
Copy link

paddle-bot-old bot commented Sep 7, 2020

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

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

Copy link
Contributor

@TCChenlong TCChenlong left a comment

Choose a reason for hiding this comment

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

LGTM

@chenwhql chenwhql merged commit 2986184 into PaddlePaddle:release/2.0-beta Sep 8, 2020
@phlrain phlrain self-requested a review September 8, 2020 02:35
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.

3 participants