Boundary check in vl_nnpool.cu about matconvnet

[vedaldi]

Hi, thanks for reporting. I believe the first version is correct as padding effectively extends the size of the image, and pooling is allowed to pool in the padded region.

On 6 Jan 2015, at 08:36, eric-phu [email protected] wrote:

Hi there,

Just want to make sure if it should be

if (data.geom.height + (padTop+padBottom) < poolHeight || data.geom.width + (padLeft+padRight) < poolWidth) {
...
}
instead of

if (data.geom.height < poolHeight || data.geom.width < poolWidth) {
...
}
at 243 in matconvnet/matlab/src/vl_nnpool.cu .

Let me know if I'm correct.

—
Reply to this email directly or view it on GitHub #36.