Skip to content

Conversation

zhaop-l
Copy link
Contributor

@zhaop-l zhaop-l commented Mar 19, 2025

Copy link

paddle-bot bot commented Mar 19, 2025

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

@CLAassistant
Copy link

CLAassistant commented Mar 19, 2025

CLA assistant check
All committers have signed the CLA.

@zhaop-l zhaop-l changed the title [Docathon][Add Inplace CN Doc No.47] [Docathon][Add Inplace CN Doc No.44、45、46、47] Mar 20, 2025
Comment on lines 236 to 238
" :ref:`paddle.tril_ <cn_api_paddle_tril_>` ", "Inplace 版本的 tril API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.triu_ <cn_api_paddle_triu_>` ", "Inplace 版本的 triu API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.trunc_ <cn_api_paddle_trunc_>` ", "Inplace 版本的 trunc API,对输入 x 采用 Inplace 策略"
Copy link
Collaborator

Choose a reason for hiding this comment

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

这些都是线性代数相关的,直接在 tensor 线性代数相关 下方新加一个表 tensor 线性代数相关原位(inplace)版本,然后把这三个 API 放进来

" :ref:`paddle.tril_ <cn_api_paddle_tril_>` ", "Inplace 版本的 tril API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.triu_ <cn_api_paddle_triu_>` ", "Inplace 版本的 triu API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.trunc_ <cn_api_paddle_trunc_>` ", "Inplace 版本的 trunc API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.where_ <cn_api_paddle_where_>` ", "Inplace 版本的 where API,对输入 x 采用 Inplace 策略"
Copy link
Collaborator

Choose a reason for hiding this comment

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

where_ 同理,是 tensor 元素查找相关的,也新加一个表

zhaop-l and others added 8 commits March 21, 2025 17:09
- 新增 tensor 元素查找相关原位(inplace)版本的 API 文档
- 新增 tensor 线性代数相关原位(inplace)版本的 API 文档
- 调整部分 API 文档的分类和顺序
- 新增 tensor 元素查找相关原位(inplace)版本的 API 文档
- 新增 tensor 线性代数相关原位(inplace)版本的 API 文档
- 调整部分 API 文档的分类和顺序
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, 修复一下 CI 检查中的 codestyle 问题,用 pre-commit 可以自动修复

@@ -211,7 +213,6 @@ tensor 数学操作原位(inplace)版本
" :ref:`paddle.gammainc_ <cn_api_paddle_gammainc_>` ", "Inplace 版本的 gammainc API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.gammaln_ <cn_api_paddle_gammaln_>` ", "Inplace 版本的 gammaln API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.gcd_ <cn_api_paddle_gcd_>` ", "Inplace 版本的 gcd API,对输入 x 采用 Inplace 策略"
" :ref:`paddle.multiply_ <cn_api_paddle_multiply_>` ", "Inplace 版本的 multiply API,对输入 x 采用 Inplace 策略"
Copy link
Collaborator

Choose a reason for hiding this comment

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

为什么要删除multiply_ ?下同。
是不是merge develop的时候操作错误。

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

Choose a reason for hiding this comment

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

不如重新开一个PR,也可以。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的

@zhaop-l zhaop-l closed this Mar 25, 2025
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.

4 participants