Skip to content

Conversation

@kexinzhao
Copy link
Contributor

fix #9142

@kexinzhao kexinzhao requested a review from chengduoZH March 16, 2018 02:41
@kexinzhao kexinzhao added the 预测 原名Inference,包含Capi预测问题等 label Mar 16, 2018
@kexinzhao kexinzhao changed the title Add float16 support for cudnn conv Add float16 support for cudnn conv2d Mar 16, 2018
REGISTER_OP_KERNEL(conv3d, CUDNN, ::paddle::platform::CUDAPlace,
REGISTER_OP_KERNEL(conv3d, CUDNN, plat::CUDAPlace,
paddle::operators::CUDNNConvOpKernel<float>,
paddle::operators::CUDNNConvOpKernel<double>);
Copy link
Contributor

Choose a reason for hiding this comment

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

The logic of conv2d and conv3d is similar, can conv3d support float16?

Copy link
Contributor

@chengduoZH chengduoZH left a comment

Choose a reason for hiding this comment

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

LGTM

@kexinzhao kexinzhao merged commit 8e73101 into PaddlePaddle:develop Mar 16, 2018
@kexinzhao kexinzhao deleted the numpy_conv2d_pool2d_fp16 branch April 4, 2018 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

预测 原名Inference,包含Capi预测问题等

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need fp16 support for conv2d op

2 participants