Skip to content
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

[Docathon][Add API Legend No.37]增加unbind图例和文档说明 #6780

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

struim
Copy link

@struim struim commented Jul 22, 2024

Copy link

paddle-bot bot commented Jul 22, 2024

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

Copy link
Collaborator

@sunzhongkai588 sunzhongkai588 Jul 25, 2024

Choose a reason for hiding this comment

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

这图片有点乱,可以简化一些,比如可以举一个 shape 为 [2, 3, 4] 的例子。绘图工具也不一定要用第三方代码库,ppt 做也可以。 能清晰展示即可

Comment on lines 12 to 13
.. image:: ../../images/api_legend/unbind.png
:alt: 图例
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
.. image:: ../../images/api_legend/unbind.png
:alt: 图例
.. image:: ../../images/api_legend/unbind.png
:width: 700
:alt: 图例

Copy link
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.

  • 感觉 3d tensor 展现的有些没那么直观,可以参考其他展现形式,如
4b49adedd9592b1a0bb3f5750498b8f5

我也简单给你画一个,比如可以参考
image

  • unbind 二维后的 tensor,用 axis = xx 代替 rows 和 columns
  • 图片尽量美观些~

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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