Skip to content

【Paddle Tensor No.21】docs: Add docs for Tensor.bitwise_invert #6932

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 2 commits into from
Nov 11, 2024
Merged

【Paddle Tensor No.21】docs: Add docs for Tensor.bitwise_invert #6932

merged 2 commits into from
Nov 11, 2024

Conversation

MrXnneHang
Copy link
Contributor

@MrXnneHang MrXnneHang commented Nov 6, 2024

PR types

New features

PR changes

Docs

Describe

Tensor.bitwise_invert 复用 Tensor.bitwise_not,两者功能用法相同。

PaddlePaddle/Paddle#69197

bitwise_invert 复用 bitwise_not,两者功能相同
Copy link

paddle-bot bot commented Nov 6, 2024

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

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 7, 2024
Out = \sim X

.. note::
``paddle.bitwise_invert`` 遵守 broadcasting,如您想了解更多,请参见 `Tensor 介绍`_ .
Copy link
Contributor

Choose a reason for hiding this comment

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

这里为什么会出现broadcasting的相关描述呢?包括英文的API docstring,bitwise系列都是一元运算,与broadcast没有任何关系吧?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://github.com/PaddlePaddle/docs/blob/develop/docs/api/paddle/bitwise_not_cn.rst

bitwise_not似乎也有一样的问题,我直接复刻过来没有仔细检查。

我要不要修改一起修改一下?以及我发现里面一些链接坏了: line 14 Tenosr介绍

Copy link
Contributor

Choose a reason for hiding this comment

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

https://github.com/PaddlePaddle/docs/blob/develop/docs/api/paddle/bitwise_not_cn.rst

bitwise_not似乎也有一样的问题,我直接复刻过来没有仔细检查。

我要不要修改一起修改一下?以及我发现里面一些链接坏了: line 14 Tenosr介绍

暂时只先考虑这个API吧,其他的API之后有时间有精力的话可以再提交一个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.

好的

@HydrogenSulfate
Copy link
Contributor

HydrogenSulfate commented Nov 7, 2024

另外这个PR得等API合入后再合入,否则CI检查无法通过

@luotao1 luotao1 changed the title 【Paddle Tensor No.23】:docs: Add docs for Tensor.bitwise_invert 【Paddle Tensor No.21】docs: Add docs for Tensor.bitwise_invert Nov 7, 2024
@luotao1 luotao1 changed the title 【Paddle Tensor No.21】docs: Add docs for Tensor.bitwise_invert 【Paddle Tensor No.21】docs: Add docs for Tensor.bitwise_invert -part Nov 7, 2024
@luotao1 luotao1 changed the title 【Paddle Tensor No.21】docs: Add docs for Tensor.bitwise_invert -part 【Paddle Tensor No.21】docs: Add docs for Tensor.bitwise_invert Nov 8, 2024
@luotao1
Copy link
Collaborator

luotao1 commented Nov 8, 2024

hi, @MrXnneHang

  • 非常感谢你对飞桨的贡献,我们正在运营一个PFCC组织,会通过定期分享技术知识与发布开发者主导任务的形式持续为飞桨做贡献,详情可见 https://github.com/luotao1 主页说明。
  • 如果你对PFCC有兴趣,请发送邮件至 ext_paddle_oss@baidu.com,我们会邀请你加入~

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 for docs

@luotao1 luotao1 merged commit 81d9b63 into PaddlePaddle:develop Nov 11, 2024
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.

4 participants