Skip to content

[Docathon][Add Inplace CN Doc No.37-38] #7113

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

Merged
merged 8 commits into from
Mar 27, 2025

Conversation

LanCole
Copy link
Contributor

@LanCole LanCole commented Mar 14, 2025

Copy link

paddle-bot bot commented Mar 14, 2025

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

@LanCole LanCole marked this pull request as draft March 15, 2025 05:39
@LanCole LanCole marked this pull request as ready for review March 15, 2025 05:39
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Mar 17, 2025
Comment on lines 12 to 26
参数
::::::::::
- **x** (Tensor) - 随机值填充的输入 Tensor。
- **mean** (float|complex|Tensor,可选) - 输出 Tensor 的正态分布的平均值,默认值为 0.0。
- **std** (float|Tensor,可选) - 输出 Tensor 的正态分布的标准差,默认值为 1.0。
- **name** (str,可选) - 具体用法请参见 :ref:`api_guide_Name`,一般无需设置,默认值为 None。

返回
::::::::::
Tensor:符合正态分布(均值为 ``mean``,标准差为 ``std`` 的正态随机分布)的随机 Tensor。

示例代码
::::::::::

COPY-FROM: paddle.normal_
Copy link
Collaborator

Choose a reason for hiding this comment

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

参数、返回等部分不用写,与其他 inplace api 文档保持一致

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的,我查看normal_ inplace api英文文档中包含参数返回内容,是在中文文档中不需要写参数/返回部分吗?
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修正,删去参数、返回等部分

Copy link
Collaborator

@sunzhongkai588 sunzhongkai588 Mar 19, 2025

Choose a reason for hiding this comment

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

好的,我查看normal_ inplace api英文文档中包含参数返回内容,是在中文文档中不需要写参数/返回部分吗? image

@BaolanChen 抱歉,这部分是我 review 的时候失误了,确实参数和非 inplace api 的不太一致,还是恢复吧


Inplace 版本的 :ref:`cn_api_paddle_normal` API,对输入 x 采用 Inplace 策略。

更多关于 inplace 操作的介绍请参考 `3.1.3 原位(Inplace)操作和非原位操作的区别`_ 了解详情。
Copy link
Collaborator

Choose a reason for hiding this comment

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

3.1.3 原位(Inplace)操作和非原位操作的区别_ 后面少了引用,参考其他的 inplace api 文档 和 如何让文档相互引用

Copy link
Contributor Author

Choose a reason for hiding this comment

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

收到,已修正

@@ -0,0 +1,26 @@
.. _cn_api_paddle_normal_:

norma\_
Copy link
Collaborator

Choose a reason for hiding this comment

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

少了 l

Copy link
Contributor Author

Choose a reason for hiding this comment

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

收到,已修正

@LanCole LanCole requested a review from sunzhongkai588 March 17, 2025 15:39
@sunzhongkai588
Copy link
Collaborator

修改一下~ #7113 (comment)

@LanCole LanCole changed the title [Docathon][Add Inplace CN Doc No.38] [Docathon][Add Inplace CN Doc No.37-38] Mar 21, 2025
@LanCole
Copy link
Contributor Author

LanCole commented Mar 21, 2025

修改一下~ #7113 (comment)

收到,已修改
同时添加了No.37 neg_中文文档内容,neg_和normal_整合在一起提交PR

@sunzhongkai588
Copy link
Collaborator

@LanCole 该 PR 合入后,可以新提一个 PR,在 Overview 里新建一个表格,即 tensor random 相关(inplace)版本, 把 normal 等相关的 inplace API 移到该表格

@LanCole LanCole closed this Mar 26, 2025
@LanCole LanCole reopened this Mar 26, 2025
@LanCole
Copy link
Contributor Author

LanCole commented Mar 26, 2025

@LanCole 该 PR 合入后,可以新提一个 PR,在 Overview 里新建一个表格,即 tensor random 相关(inplace)版本, 把 normal 等相关的 inplace API 移到该表格

收到,我具体看一下怎么做

@luotao1
Copy link
Collaborator

luotao1 commented Mar 27, 2025

@LanCole 请解决下冲突

Copy link
Contributor Author

@LanCole LanCole left a comment

Choose a reason for hiding this comment

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

解决冲突,分支合并

@LanCole
Copy link
Contributor Author

LanCole commented Mar 27, 2025

@LanCole 请解决下冲突

收到,已解决代码冲突问题

@luotao1 luotao1 merged commit a4238a3 into PaddlePaddle:develop Mar 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants