Possible invalid memory access? about grenade HOT 2 CLOSED

im2col isn't a total function, it only works where the kernel and stride "fit" for the input the matrix. So yes, it does look like that's what's happening.

This "fitting" constraint is captured in the proofs going into the convolution and deconvolution layers. I found that if I tried to push these through to the im2col function it lead to a considerable slowdown, so put them in an internal function set instead.

It could probably be better documented, and maybe even just stop with error inside im2col if these constraints are violated.

from grenade.

Yeah, it seems like either erroring out in im2col when the arguments aren't valid would be a good safety net.

from grenade.