Padding is eaten away

[torch]

Padding is eaten away

[soumith]

padding is eaten away and replaced by .padW and .padH

[soumith]

torch/nn@793b6bf

[Atcold]

OK, but I get the error. How can I forward the image successfully?
The error message is not really helpful...
Should it be ".padW and .padH are missing"?

[soumith]

did you update cunn to the latest trunk? seems like you updated nn, but not cunn?

[soumith]

https://github.com/torch/cunn/blob/master/SpatialConvolutionMM.cu#L111

[Atcold]

Everything is just to the latest release.
OK, tnx for the pointer, but I think there should be a different error message. Don't you think so?

[soumith]

i think that has to be double-checked.

bad argument #1 (field padding does not exist)

This error appears on the older version of cunn for sure, because of this line:

cunn/SpatialConvolutionMM.cu

Line 124 in 30e394d

int padding = luaT_getfieldcheckint(L, 1, "padding");

but should not appear in trunk, because it is changed to:
https://github.com/torch/cunn/blob/master/SpatialConvolutionMM.cu#L111

[Atcold]

@soumith, my bad. apt-get broke cuda and luarocks didn't update cunn and I didn't notice that. My fault. Sorry...

[soumith]

no problem :)

[Atcold]

👍

[Atcold]

@soumith, one more note. With a broken cuda, luarocks installed only the lua files leaving out the cu one. That's why I had the lua killing my padding variable but the precompiled cu one asking for it. So, the semi-installed cunn gave me some weird troubles.