Skip to content

[Docathon][Update Doc No.31] update the api path #7228

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 13 commits into from
Apr 21, 2025

Conversation

Echo-Nie
Copy link
Contributor

@Echo-Nie Echo-Nie commented Apr 4, 2025

Copy link

paddle-bot bot commented Apr 4, 2025

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

@Echo-Nie
Copy link
Contributor Author

Echo-Nie commented Apr 4, 2025

@DrRyanHuang pls review

Copy link
Collaborator

@DrRyanHuang DrRyanHuang left a comment

Choose a reason for hiding this comment

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

@Echo-Nie Echo-Nie requested a review from DrRyanHuang April 7, 2025 12:35
Copy link
Collaborator

@DrRyanHuang DrRyanHuang left a comment

Choose a reason for hiding this comment

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

格式没啥大问题了,将参数对齐一下(英文版也改一下)


- ``pooled_height`` 和 ``pooled_width``\ : 这里可以接受非正方的池化窗口大小
- ``boxes``\ : 接收\ ``DenseTensor``\ 类型来表示需要池化的 Regions of Interest,关于 RoI 的解释请参考\ `论文 <https://arxiv.org/abs/1506.01497>`__
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
- ``boxes``\ : 接收\ ``DenseTensor``\ 类型来表示需要池化的 Regions of Interest,关于 RoI 的解释请参考\ `论文 <https://arxiv.org/abs/1506.01497>`__
- ``boxes``\ : 接收\ ``Tensor``\ 类型来表示需要池化的 Regions of Interest,关于 RoI 的解释请参考\ `论文 <https://arxiv.org/abs/1506.01497>`__

- ``rois`` : It receives ``DenseTensor`` type to indicate the Regions of Interest that needs to be pooled. For an explanation of RoI, please refer to `Paper <https://arxiv.org/abs/1506.01497>`__
- ``x`` : The input feature map, with a shape of (N, C, H, W).

- ``boxes`` : It receives ``DenseTensor`` type to indicate the Regions of Interest that needs to be pooled. For an explanation of RoI, please refer to `Paper <https://arxiv.org/abs/1506.01497>`__
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
- ``boxes`` : It receives ``DenseTensor`` type to indicate the Regions of Interest that needs to be pooled. For an explanation of RoI, please refer to `Paper <https://arxiv.org/abs/1506.01497>`__
- ``boxes`` : It receives ``Tensor`` type to indicate the Regions of Interest that needs to be pooled. For an explanation of RoI, please refer to `Paper <https://arxiv.org/abs/1506.01497>`__


- ``return_mask``\ : 是否返回最大索引和输出。
Copy link
Collaborator

Choose a reason for hiding this comment

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

加一下仅最大值池化有这个参数,英文版也加一下

Copy link
Collaborator

@DrRyanHuang DrRyanHuang left a comment

Choose a reason for hiding this comment

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

LGTM

@luotao1 luotao1 merged commit b6d2902 into PaddlePaddle:develop Apr 21, 2025
2 checks passed
@Echo-Nie Echo-Nie deleted the No31 branch April 22, 2025 02:44
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