Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Docathon][Add API Legend No.26~30] add tensor_split series API legend #66991

Merged
merged 2 commits into from
Aug 19, 2024

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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants