Skip to content

Conversation

@AnnaTrainingG
Copy link
Contributor

@AnnaTrainingG AnnaTrainingG commented Feb 8, 2022

PR types

Others

PR changes

OPs

Describe

Delete BASE_SIZE in elementwise_base.h

本地性能测试结果如下:

op name axise dtype shape dev (us) new (us)  
sum [1] float32 [16, 8, 128] 1.546 1.546 1.00
sum backward [1] float32 [16, 8, 128] 2.783 2.83 1.02
prod [1] float32 [16, 8, 128] 1.56 1.599 1.03
prod backward [1] float32 [16, 8, 128] 4.496 4.491 1.00

case 比较小 机器波动影响比较大。

@paddle-bot-old
Copy link

paddle-bot-old bot commented Feb 8, 2022

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

Copy link
Contributor

@qili93 qili93 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
Collaborator

@sneaxiy sneaxiy 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

@Xreki Xreki left a comment

Choose a reason for hiding this comment

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

Good work~

Copy link
Contributor

@ZzSean ZzSean 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 ci op benchmark

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

Labels

None yet

7 participants