Skip to content

Conversation

fufu0615
Copy link
Contributor

@fufu0615 fufu0615 commented Aug 4, 2024

PR Category

User Experience

PR Types

Docs

Description

修改tensor_split,hsplit,dsplit,vsplit英文文档描述并增加图例
(图例的url需要等中文文档pr合并后才能访问)

对应的中文文档修复为:PaddlePaddle/docs#6809

Copy link

paddle-bot bot commented Aug 4, 2024

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

Comment on lines 3059 to 3098
``vsplit`` Full name Vertical Split, splits the input Tensor into multiple sub-Tensors along the vertical axis, which is equivalent to ``paddle.tensor_split`` with ``axis=0``.
1. When the number of Tensor dimensions is equal to 1:
.. image:: https://githubraw.cdn.bcebos.com/PaddlePaddle/docs/develop/docs/images/api_legend/vsplit/vsplit-1.png
2. When the number of Tensor dimensions is greater than 1:
.. image:: https://githubraw.cdn.bcebos.com/PaddlePaddle/docs/develop/docs/images/api_legend/vsplit/vsplit-2.png
Copy link
Contributor

Choose a reason for hiding this comment

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

维度不能小于 2


>>> import paddle

>>> # split along axis with indice
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
>>> # split along axis with indice
>>> # split along axis with indices

@fufu0615 fufu0615 force-pushed the develop branch 2 times, most recently from 5ea2bee to fb8612c Compare August 8, 2024 03:36
@fufu0615 fufu0615 requested a review from megemini August 8, 2024 03:45
Copy link
Contributor

@megemini megemini left a comment

Choose a reason for hiding this comment

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

vsplit 没有修改

Copy link
Contributor

@megemini megemini left a comment

Choose a reason for hiding this comment

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

LGTM ~

预览看不到图片,需要合入才能看到?

@fufu0615
Copy link
Contributor Author

fufu0615 commented Aug 8, 2024

LGTM ~

预览看不到图片,需要合入才能看到?

可能要等中文文档那边的pr合入吧,因为图例是存在docs仓库下面的

Copy link

paddle-ci-bot bot commented Aug 16, 2024

Sorry to inform you that 9cd20ad's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@luotao1 luotao1 merged commit f15bc54 into PaddlePaddle:develop Aug 19, 2024
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants