{"action":"created","issue":{"url":"https://api.github.com/repos/torch/nn/issues/481","labels_url":"https://api.github.com/repos/torch/nn/issues/481/labels{/name}","comments_url":"https://api.github.com/repos/torch/nn/issues/481/comments","events_url":"https://api.github.com/repos/torch/nn/issues/481/events","html_url":"https://github.com/torch/nn/pull/481","id":117445659,"number":481,"title":"Added Padding option in Volumetric convolution","user":{"login":"kmul00","id":6665222,"avatar_url":"https://avatars.githubusercontent.com/u/6665222?v=3","gravatar_id":"","url":"https://api.github.com/users/kmul00","html_url":"https://github.com/kmul00","followers_url":"https://api.github.com/users/kmul00/followers","following_url":"https://api.github.com/users/kmul00/following{/other_user}","gists_url":"https://api.github.com/users/kmul00/gists{/gist_id}","starred_url":"https://api.github.com/users/kmul00/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kmul00/subscriptions","organizations_url":"https://api.github.com/users/kmul00/orgs","repos_url":"https://api.github.com/users/kmul00/repos","events_url":"https://api.github.com/users/kmul00/events{/privacy}","received_events_url":"https://api.github.com/users/kmul00/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-11-17T20:42:20Z","updated_at":"2015-11-24T17:51:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/torch/nn/pulls/481","html_url":"https://github.com/torch/nn/pull/481","diff_url":"https://github.com/torch/nn/pull/481.diff","patch_url":"https://github.com/torch/nn/pull/481.patch"},"body":"**VolumetricConvolution** didn't support ***padding*** before. With this PR ***padding*** can be provided in **VolumetricConvolution** in a way similar to **SpatialConvolution**. That is,\r\n\r\n    module = nn.VolumetricConvolution(nInputPlane, nOutputPlane, kT, kW, kH, [dT], [dW], [dH], [padT], [padW], [padH])\r\n\r\nBy default `padT=padW=padH=0`\r\n\r\nTo be able to add ***padding*** support I had to write a new module to perform Volumetric Convolution, `VolumetricConvolutionMM.c` which, similar to `SpatialConvolutionMM`, constructs Toeplitz matrix and does matrix multiplications. This is done to keep the current implementation of **VolumetricConvolution** intact (`VolumetricConvolution.c`) and it was difficult to add this functionality in it.\r\n\r\nI have added necessary test cases in `test.lua`  and initializations in `init.c` as well.\r\n\r\nI will be adding similar functionality to the `cuda` version in `cunn`."},"comment":{"url":"https://api.github.com/repos/torch/nn/issues/comments/159354525","html_url":"https://github.com/torch/nn/pull/481#issuecomment-159354525","issue_url":"https://api.github.com/repos/torch/nn/issues/481","id":159354525,"user":{"login":"kmul00","id":6665222,"avatar_url":"https://avatars.githubusercontent.com/u/6665222?v=3","gravatar_id":"","url":"https://api.github.com/users/kmul00","html_url":"https://github.com/kmul00","followers_url":"https://api.github.com/users/kmul00/followers","following_url":"https://api.github.com/users/kmul00/following{/other_user}","gists_url":"https://api.github.com/users/kmul00/gists{/gist_id}","starred_url":"https://api.github.com/users/kmul00/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kmul00/subscriptions","organizations_url":"https://api.github.com/users/kmul00/orgs","repos_url":"https://api.github.com/users/kmul00/repos","events_url":"https://api.github.com/users/kmul00/events{/privacy}","received_events_url":"https://api.github.com/users/kmul00/received_events","type":"User","site_admin":false},"created_at":"2015-11-24T17:51:00Z","updated_at":"2015-11-24T17:51:00Z","body":"Thanks a lot."}}