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

fast Depthwise mxnet operator #8

Closed
vsooda opened this issue Aug 7, 2017 · 11 comments
Closed

fast Depthwise mxnet operator #8

vsooda opened this issue Aug 7, 2017 · 11 comments

Comments

@vsooda
Copy link

vsooda commented Aug 7, 2017

@KeyKy Would you mind update the fast depthwise operator. or could you describe the key for implement fast operator. Thank you.

@KeyKy
Copy link
Owner

KeyKy commented Aug 7, 2017

@vsooda fast depthwise operator is implemented by just copy the cuda code from ^-^ url. And then you can get the 2ms/image.

@KeyKy KeyKy closed this as completed Aug 7, 2017
@ccJia
Copy link

ccJia commented Aug 7, 2017

@KeyKy Have some good idea or implement on CPU T-T ??

@KeyKy
Copy link
Owner

KeyKy commented Aug 7, 2017

@ccJia fast depthwise operator (which num_group == num_outputs) should be calculated in multi-threaded channel-by-channel. According to tencent's ncnn, I think sliding window (highly optimized, which also using in BVLC/caffe#5665) maybe faster than im2col+GEMM.

@ccJia
Copy link

ccJia commented Aug 7, 2017

@KeyKy OK, thank you . Let me have a try ^-^.

@KeyKy
Copy link
Owner

KeyKy commented Aug 7, 2017

@ccJia looking forward to your job. Could you share with me after you have done the work? ^-^

@ccJia
Copy link

ccJia commented Aug 7, 2017

@KeyKy Some of my friends told me that the direct convolution like NCNN is not work.....

@KeyKy
Copy link
Owner

KeyKy commented Aug 7, 2017

@ccJia Did they explain the reason or they try?

@ccJia
Copy link

ccJia commented Aug 7, 2017

@KeyKy They tried on Win.

@KeyKy
Copy link
Owner

KeyKy commented Aug 7, 2017

@ccJia OK. I will try it in my spare time.

@ccJia
Copy link

ccJia commented Aug 7, 2017

@KeyKy waiting your good news .

@vsooda
Copy link
Author

vsooda commented Aug 7, 2017

@KeyKy thank you

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

3 participants