Skip to content

Conversation

AnnaTrainingG
Copy link
Contributor

@AnnaTrainingG AnnaTrainingG commented Jan 13, 2022

PR types

Others

PR changes

OPs

Describe

Modified Reduce for XPU2
修改点:
1.将Reduce中的XPU的block size修改为: 64 (XPU2最大只支持64)
2.Reduce any在case较大时会出现计算错误的情况,目前已经修复

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot-old
Copy link

paddle-bot-old bot commented Feb 2, 2022

Sorry to inform you that b1d0000's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

const std::vector<int>& cal_dims,
const std::vector<int>& cal_strides,
const std::vector<int>& full_strides)
: dim(dim) {
Copy link
Contributor

Choose a reason for hiding this comment

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

在 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.

好的,已经添加

@AnnaTrainingG AnnaTrainingG merged commit 909d1e6 into PaddlePaddle:develop Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants