Skip to content

【Hackathon 5th No.6】 为 Paddle 增强put_along_axis API#6348

Merged
luotao1 merged 3 commits into
PaddlePaddle:developfrom
YibinLiu666:put_along_axis
Dec 14, 2023
Merged

【Hackathon 5th No.6】 为 Paddle 增强put_along_axis API#6348
luotao1 merged 3 commits into
PaddlePaddle:developfrom
YibinLiu666:put_along_axis

Conversation

@YibinLiu666

Copy link
Copy Markdown
Contributor

黑客松第六题:PaddlePaddle/Paddle#59674
以及快乐开源 :PaddlePaddle/Paddle#59163
相关改动的中文doc

@jeff41404 jeff41404 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread docs/api/paddle/Tensor_cn.rst Outdated
:::::::::

基于输入 indices 矩阵,将输入 value 沿着指定 axis 放置入 arr 矩阵。索引矩阵和 value 必须和 arr 矩阵有相同的维度,需要能够 broadcast 与 arr 矩阵对齐。
基于输入 indices 矩阵,将输入 value 沿着指定 axis 放置入 arr 矩阵。索引矩阵和 value 必须和 arr 矩阵有相同的维度,如果 ``broadcast``为 ``True``,则需要能够 broadcast 与 arr 矩阵对齐。

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
基于输入 indices 矩阵,将输入 value 沿着指定 axis 放置入 arr 矩阵。索引矩阵和 value 必须和 arr 矩阵有相同的维度,如果 ``broadcast``为 ``True``,则需要能够 broadcast 与 arr 矩阵对齐。
基于输入 indices 矩阵,将输入 value 沿着指定 axis 放置入 arr 矩阵。索引矩阵和 value 必须和 arr 矩阵有相同的维度,如果 ``broadcast`` 为 ``True``,则需要能够 broadcast 与 arr 矩阵对齐。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done, 同时新增了一些参数映射表的相关doc,麻烦孙哥再看看

@sunzhongkai588 sunzhongkai588 left a comment

Copy link
Copy Markdown
Collaborator

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 ca33083 into PaddlePaddle:develop Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor PaddlePaddle Hackathon 飞桨黑客松活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants