Skip to content
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

Need to add some new algorithm(like direct, winograd) for convolution calculation. #2177

Closed
hedaoyuan opened this issue May 17, 2017 · 2 comments
Assignees

Comments

@hedaoyuan
Copy link
Contributor

At present, there are only sgemm and CUDNN-based convolution implementations in Paddle. In a model training or prediction without a GPU, can only select convolution calculations based on sgemm. However, based on the sgemm convolution calculation, performance is not optimal in many scenarios. see here

We also encountered the problem of convolution computing performance when deploying Paddle into some product environments. At the same time, there are many excellent convolution implementation libraries, I think we can try to import it into Paddle to improve the Paddle convolution calculation performance.

@hedaoyuan hedaoyuan self-assigned this May 17, 2017
This was referenced May 17, 2017
@hedaoyuan
Copy link
Contributor Author

Convolution performance

Convolution layer performance in a ResNet model with 3x3 kernel. Test in a raspberry environment, and build with NNPACK. We can see that some layers with gemm algorithm performance better, while some layers with wt8x8 algorithm performance better.

image

@Xreki
Copy link
Contributor

Xreki commented May 14, 2018

NNPACK is supported in v2 Paddle and new features won't be implemented based on v2, so close this issue.

@Xreki Xreki closed this as completed May 14, 2018
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

No branches or pull requests

2 participants