clementfarabet / torch-ios

Torch7 for iOS.
Other
196 stars 42 forks source link

Padding in SpatialConvolution? #32

Closed PhillipZeratul closed 7 years ago

PhillipZeratul commented 7 years ago

It seems SpatialConvolution does not have padding functionality. So when I use torch-ios to load a torch7 trained model, the result is incorrect.

Is there a way to fix this?

Here is what I see in torch-ios (SpatialConvolution.lua): function SpatialConvolution:__init(nInputPlane, nOutputPlane, kW, kH, dW, dH)

Here is what it is like in torch7: module = nn.SpatialConvolution(nInputPlane, nOutputPlane, kW, kH, [dW], [dH], [padW], [padH])

liamnaka commented 7 years ago

As stated by @soumith , this library is quite outdated. Many of the torch methods are missing newer components.

logic1988 commented 7 years ago

Yes. I have the same problem. Is there a new solution? Is the torch official provided the method to generate IOS library?

EliseSo commented 7 years ago

same problem here, can't find a proper solution