-
Notifications
You must be signed in to change notification settings - Fork 5.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move bn to pten #39347
Move bn to pten #39347
Conversation
Thanks for your contribution! |
d4b0fe7
to
2b4a9eb
Compare
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
|
||
DenseTensor dy_sum; | ||
dy_sum.Resize({C}); | ||
dy_sum.mutable_data<T>(ctx.GetPlace()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好像还有一些mutable_data,需要替换成ctx.Alloc,建议全局搜索替换一下
#endif | ||
|
||
#ifdef PADDLE_WITH_HIP | ||
PD_REGISTER_KERNEL(batch_norm_grad_grad, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个命名我们后续需要统一一下写法
"But received: the size of input's dimensions is [%d]", | ||
x_dims.size())); | ||
|
||
y->mutable_data<T>(ctx.GetPlace()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mutable_data -> Alloc
… move_bn_to_pten
… move_bn_to_pten
… move_bn_to_pten
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR types
Breaking changes
PR changes
OPs
Describe
move batch norm to pten