-
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
Move out sequential and replace save_dygraph and load_dygraph #48709
Move out sequential and replace save_dygraph and load_dygraph #48709
Conversation
…paddle.load instead
… remove_save_dygraph_and_load_dygraph
… remove_save_dygraph_and_load_dygraph
… move_out_sequential
你的PR提交成功,感谢你对开源项目的贡献! |
… move_out_sequential
… move_out_sequential
… move_out_sequential
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 for PADDLE_NEED_CONVERT_APIS
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 for rm parallel_UT
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 for API migration
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
PR types
Others
PR changes
APIs
Describe
because of #48477, we don't move out the
sequrntial
from fluid, nowthe dynamic to static has been migrated, so the sequential is migrated to paddle.nn
And the migration of Sequential depends on the removal of save_dygraph and load_dygraph, so submit the two together