Fast-CNN icon indicating copy to clipboard operation
Fast-CNN copied to clipboard

Claim is misleading. CNN is not slower. The CNN implementation is slow.

Open geyang opened this issue 4 years ago • 1 comments

The claim in this article and the notebook is quite misleading.

pyTorch CNN implementation already uses the cuDNN or a faster fbfft module. For smaller kernel sizes, the Winograd algorithm is even faster.

Your comparison is against convolution done in the position space, using a python for loop.

geyang avatar Apr 24 '21 21:04 geyang

pyTorch CNN implementation already uses the cuDNN or a faster fbfft module

@geyang. Could you link that code from pytorch's github? I am very interested in that and I couldn't find that yet.

eduardo4jesus avatar Jun 22 '21 06:06 eduardo4jesus