Skip to content

[Docathon][Fix note No.20-22] #7148

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
Closed
wants to merge 4 commits into from
Closed

[Docathon][Fix note No.20-22] #7148

wants to merge 4 commits into from
+19 −17

Conversation

594233
Copy link
Contributor

@594233 594233 commented Mar 24, 2025

Indent the command content of the following three RSTs:

@594233
Indent the command content of the following three RSTs-docs/api/paddl…
615614e 
…e/optimizer/NAdam_cn.rst-Optimizer_cn.rst-RAdam_cn.rst
@CLAassistant
Copy link

CLAassistant commented Mar 24, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@paddle-bot Paddle-bot
Copy link

paddle-bot bot commented Mar 24, 2025

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

@luotao1 luotao1 mentioned this pull request Mar 25, 2025
Closed
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Mar 25, 2025
Echo-Nie
Echo-Nie reviewed Mar 25, 2025
View reviewed changes
docs/api/paddle/Overview_cn.rst Outdated
" :ref:`paddle.baddbmm_ <cn_api_paddle_log_>` ", "Inplace 版本的 log API，对输入 x 采用 Inplace 策略"
" :ref:`paddle.baddbmm_ <cn_api_paddle_log10_>` ", "Inplace 版本的 log10 API，对输入 x 采用 Inplace 策略"
" :ref:`paddle.baddbmm_ <cn_api_paddle_log2_>` ", "Inplace 版本的 log2 API，对输入 x 采用 Inplace 策略"
" :ref:`paddle.baddbmm_ <cn_api_paddle_cast_>` ", "Inplace 版本的 cast API，对输入 x 采用 Inplace 策略"
Copy link
Contributor

Choose a reason for hiding this comment

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

The API and ref don't match

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your comment, I have corrected these errors.

@Echo-Nie
Copy link
Contributor

Echo-Nie commented Mar 25, 2025 •
edited
Loading

PR seems a bit confusing? It seems to involve the modification of [Overview_cn.rst], but we only include the follows:

  • docs/api/paddle/optimizer/NAdam_cn.rst
  • docs/api/paddle/optimizer/Optimizer_cn.rst
  • docs/api/paddle/optimizer/RAdam_cn.rst

Overview_cn.rst should be in another task: #7090
I see that you also signed up for the 7090issue, if the PR is wrong, can close the PR and resubmit a new PR

@594233
Copy link
Contributor Author

594233 commented Mar 25, 2025

PR seems a bit confusing? It seems to involve the modification of [Overview_cn.rst], but we only include the follows:

  • docs/api/paddle/optimizer/NAdam_cn.rst
  • docs/api/paddle/optimizer/Optimizer_cn.rst
  • docs/api/paddle/optimizer/RAdam_cn.rst

Overview_cn.rst should be in another task: #7090 I see that you also signed up for the 7090issue, if the PR is wrong, can close the PR and resubmit a new PR

OK, thank you for reminding me.

@594233 594233 closed this Mar 25, 2025
@594233 594233 deleted the my-cool-stuff branch March 25, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Echo-Nie Echo-Nie

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.

None yet
5 participants
@594233 @CLAassistant @Echo-Nie @luotao1 @sunzhongkai588