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.

Sign up for GitHub

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

Jump to bottom

[Docathon] Add paddle.incubate.ampdecorate and paddle.incubate.amp.prune_model API CN Doc #6222

Merged
merged 15 commits into from
Oct 16, 2023
Merged

[Docathon] Add paddle.incubate.ampdecorate and paddle.incubate.amp.prune_model API CN Doc #6222

merged 15 commits into from
Oct 16, 2023

Conversation

ooooo-create
Copy link
Collaborator

@ooooo-create ooooo-create commented Oct 9, 2023 •
edited
Loading

@paddle-bot
Copy link

paddle-bot bot commented Oct 9, 2023

感谢你贡献飞桨文档，文档预览构建中，Docs-New 跑完后即可预览，预览链接：http://preview-pr-6222.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明，请参考：飞桨文档预览工具

@ooooo-create ooooo-create changed the title [Docathon] Add paddle.incubate.decorate and paddle.incubate.prune_model API CN Doc [Docathon] Add paddle.incubate.ampdecorate and paddle.incubate.amp.prune_model API CN Doc Oct 9, 2023
@ooooo-create ooooo-create mentioned this pull request Oct 10, 2023
Merged
SigureMo
SigureMo reviewed Oct 10, 2023
View reviewed changes
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

另外需要在 PR 描述里添加 PADDLEPADDLE_PR=57973 以确保构建的预览是使用 PaddlePaddle/Paddle#57973 里 CI build 的 wheel 包

docs/api/paddle/incubate/amp/decorate_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/amp/prune_model_cn.rst Outdated Show resolved Hide resolved
@SigureMo
Copy link
Member

另外需要在 PR 描述里添加 PADDLEPADDLE_PR=57973 以确保构建的预览是使用 PaddlePaddle/Paddle#57973 里 CI build 的 wheel 包

咦，等一下，这个 PR 和 PaddlePaddle/Paddle#57973 之间关系是什么？这个 PR 不依赖 PaddlePaddle/Paddle#57973 的话可以不用加

@ooooo-create
Copy link
Collaborator Author

另外需要在 PR 描述里添加 PADDLEPADDLE_PR=57973 以确保构建的预览是使用 PaddlePaddle/Paddle#57973 里 CI build 的 wheel 包

咦，等一下，这个 PR 和 PaddlePaddle/Paddle#57973 之间关系是什么？这个 PR 不依赖 PaddlePaddle/Paddle#57973 的话可以不用加

我需分别描述动态图和静态图两个模式的代码，依赖那个的

SigureMo
SigureMo reviewed Oct 10, 2023
View reviewed changes
docs/api/paddle/incubate/amp/decorate_cn.rst Outdated
COPY-FROM paddle.incubate.amp.decorate
1. 动态图模式

COPY-FROM: paddle.incubate.amp.decorate:code-example1
Copy link
Member

Choose a reason for hiding this comment

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

如果明确是动态图和静态图这么明确的语义的话，建议使用明确的 dygraph 和 static 标签，code-example1 这种模板化的标签是不推荐的

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

sunzhongkai588
sunzhongkai588 reviewed Oct 11, 2023
View reviewed changes
docs/api/paddle/incubate/asp/prune_model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/asp/prune_model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/asp/prune_model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/asp/prune_model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/asp/prune_model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/asp/prune_model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/asp/prune_model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/incubate/asp/decorate_cn.rst Outdated Show resolved Hide resolved
@sunzhongkai588 sunzhongkai588 added the HappyOpenSource 快乐开源活动issue与PR label Oct 11, 2023
ooooo-create and others added 9 commits October 11, 2023 11:58
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/prune_model_cn.rst
c72c883 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/prune_model_cn.rst
8843386 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/decorate_cn.rst
09c9d8f 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/prune_model_cn.rst
5313fc5 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/prune_model_cn.rst
ec63520 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/prune_model_cn.rst
e7ef60c 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/prune_model_cn.rst
fd126fd 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create @sunzhongkai588
Update docs/api/paddle/incubate/asp/prune_model_cn.rst
8514ad5 
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
@ooooo-create
fix spell
7227040
sunzhongkai588
sunzhongkai588 approved these changes Oct 16, 2023
View reviewed changes
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

@luotao1 luotao1 merged commit 67577d1 into PaddlePaddle:develop Oct 16, 2023
1 check passed
@ooooo-create ooooo-create deleted the ooooo/add_amp_api_cn_doc branch October 16, 2023 09:53
@sunzhongkai588 sunzhongkai588 mentioned this pull request Oct 17, 2023
Closed
@ooooo-create ooooo-create mentioned this pull request Oct 30, 2023
Closed
@ooooo-create ooooo-create mentioned this pull request Dec 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SigureMo SigureMo SigureMo left review comments

@sunzhongkai588 sunzhongkai588 sunzhongkai588 approved these changes

Assignees

@luotao1 luotao1

@sunzhongkai588 sunzhongkai588

Labels
contributor HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ooooo-create @SigureMo @sunzhongkai588 @luotao1