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

remove with_data_parallel in doc of custom_op #5766

Conversation

kangguangli
Copy link
Contributor

API with_data_parallel 已经从Paddle develop分支删除,本PR更新自定义算子的文档,将静态图示例代码从单机多卡改为单卡。后续分布式训练应使用fleetAPI。

相关PR

PaddlePaddle/Paddle#50425

@@ -1753,9 +1753,7 @@ exe = static.Executor()
exe.run(static.default_startup_program())

places = paddle.static.cuda_places()
compiled_program = static.CompiledProgram(
Copy link
Contributor

Choose a reason for hiding this comment

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

places = paddle.static.cuda_places()

Maybe this line could also be deleted?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

@jiahy0825 jiahy0825 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
Collaborator

@sunzhongkai588 sunzhongkai588 left a comment

Choose a reason for hiding this comment

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

LGTM

@sunzhongkai588 sunzhongkai588 merged commit 053536a into PaddlePaddle:develop Mar 31, 2023
@kangguangli kangguangli deleted the remove_with_data_parallel_in_doc branch April 3, 2023 09:33
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