Skip to content

Conversation

@BrilliantYuKaimin
Copy link
Contributor

@BrilliantYuKaimin BrilliantYuKaimin commented Apr 8, 2022

增加nn.ChannelShuffle和nn.functional.channel_shuffle说明文档。

该算子实现的PR为PaddlePaddle/Paddle#40743
PADDLEPADDLE_PR=40743

@paddle-bot-old
Copy link

paddle-bot-old bot commented Apr 8, 2022

Thanks for your contribution!

RichardWooSJTU pushed a commit to RichardWooSJTU/docs that referenced this pull request Apr 8, 2022
@dingjiaweiww dingjiaweiww self-requested a review April 12, 2022 03:37
形状
:::::::::
- **x** (Tensor): 默认形状为 (批大小,通道数,高度,宽度),即NCHW格式的4-D Tensor。其数据类型为float32, float64;
- **output** (Tensor): 其形状与数据类型均和输入相同。
Copy link
Collaborator

Choose a reason for hiding this comment

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

image

「:」--> 「-」

注意标点符号的正确使用~ 参考文档

# [[0.40000001]],
# [[0.10000000]],
# [[0.30000001]],
# [[0.50000000]]]]
Copy link
Collaborator

Choose a reason for hiding this comment

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

代码示例建议使用copy-from的方式,保持和英文的一致,文档参考

- **x** (Tensor): 当前算子的输入,其是一个形状为 `[N, C, H, W]` 的4-D Tensor。其中 `N` 是batch size,`C` 是通道数,`H` 是输入特征的高度,`W` 是输入特征的宽度。其数据类型为float32或者float64;
- **groups** (int): 要把通道分成的组数;
- **data_format** (str,可选): 数据格式,可选: "NCHW"或"NHWC",默认: "NCHW";
- **name** (str,可选): 操作的名称(可选,默认值为None)。更多信息请参见 :ref:`api_guide_Name`。
Copy link
Collaborator

Choose a reason for hiding this comment

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

同ChannelShuffle,「:」--> 「-」

# [[0.40000001]],
# [[0.10000000]],
# [[0.30000001]],
# [[0.50000000]]]]
Copy link
Collaborator

Choose a reason for hiding this comment

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

代码示例建议使用copy-from的方式,保持和英文的一致

Copy link
Collaborator

@Ligoml Ligoml left a comment

Choose a reason for hiding this comment

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

LGTM

@Ligoml Ligoml merged commit 9e8b71c into PaddlePaddle:develop Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants