prdwb/cuda-convnet

Confirmation for strange-looking assert in filter_acts.cu

Closed this issue · 1 comments

Hi Alex,

Could you confirm that this slightly strange-looking line from filter_act.cu is 
correct?

assert(paddingStart <= 0 && paddingStart + (numModules-1)*moduleStride + 
filterSize >= imgSize);

In particular, why are you multiplying numModules (which the square of 
numModulesX) by the stride and adding filterSize (which is not the square, but 
the side-length).

If you're sure, I trust you, but if you could additionally lend some intuition 
for why I'd appreciate it.

There is a comment in the code but I still don't get it:
// These routines don't handle the case when only part of the image is visited 
in the convolution

Thanks,
- James

Original issue reported on code.google.com by james.be...@gmail.com on 10 Jan 2012 at 10:35

Original comment by akrizhev...@gmail.com on 13 Jan 2012 at 3:23

  • Changed state: Fixed