Skip to content

Conversation

@paddle-bot
Copy link

paddle-bot bot commented May 8, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot
Copy link

paddle-bot bot commented May 8, 2023

❌ The PR is not created using PR's template. You can refer to this Demo.
Please use PR's template, it helps save our maintainers' time so that more developers get helped.

@zhwesky2010 zhwesky2010 changed the title 0d 2.5 [cherry-pick 2.5][Zero-Dim] support paddle.sum/mean/loass api output 0D May 8, 2023
Co-authored-by: zhwesky2010 <zhouwei25@baidu.com>
Copy link
Contributor

@jeff41404 jeff41404 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@From00 From00 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 use mutable_data

@zhwesky2010 zhwesky2010 changed the title [cherry-pick 2.5][Zero-Dim] support paddle.sum/mean/loass api output 0D [cherry-pick 2.5][Zero-Dim] support paddle.sum/mean/loss api output 0D May 8, 2023
Copy link
Contributor

@hong19860320 hong19860320 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@cyber-pioneer cyber-pioneer 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 composite rule

@XiaoguangHu01 XiaoguangHu01 merged commit b6e2377 into PaddlePaddle:release/2.5 May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

9 participants