{"action":"created","issue":{"url":"https://api.github.com/repos/torch/cunn/issues/143","labels_url":"https://api.github.com/repos/torch/cunn/issues/143/labels{/name}","comments_url":"https://api.github.com/repos/torch/cunn/issues/143/comments","events_url":"https://api.github.com/repos/torch/cunn/issues/143/events","html_url":"https://github.com/torch/cunn/pull/143","id":108033272,"number":143,"title":"Fast SpatialBatchNormalization (Facebook)","user":{"login":"szagoruyko","id":4953728,"avatar_url":"https://avatars.githubusercontent.com/u/4953728?v=3","gravatar_id":"","url":"https://api.github.com/users/szagoruyko","html_url":"https://github.com/szagoruyko","followers_url":"https://api.github.com/users/szagoruyko/followers","following_url":"https://api.github.com/users/szagoruyko/following{/other_user}","gists_url":"https://api.github.com/users/szagoruyko/gists{/gist_id}","starred_url":"https://api.github.com/users/szagoruyko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/szagoruyko/subscriptions","organizations_url":"https://api.github.com/users/szagoruyko/orgs","repos_url":"https://api.github.com/users/szagoruyko/repos","events_url":"https://api.github.com/users/szagoruyko/events{/privacy}","received_events_url":"https://api.github.com/users/szagoruyko/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-24T00:28:42Z","updated_at":"2015-09-24T00:58:32Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/torch/cunn/pulls/143","html_url":"https://github.com/torch/cunn/pull/143","diff_url":"https://github.com/torch/cunn/pull/143.diff","patch_url":"https://github.com/torch/cunn/pull/143.patch"},"body":"An amazing work of @nicolasvasilache , fast and no extra memory.\r\nIn fact I ported it more than a week ago and couldn't find time to factor out c++11 and sm_35. So to be added:\r\n - [ ] remove c++11\r\n - [ ] put a condition on sm_35 or find a workaround\r\n - [ ] port test against CPU version\r\nCPU patch and BatchNormalization are coming too."},"comment":{"url":"https://api.github.com/repos/torch/cunn/issues/comments/142771794","html_url":"https://github.com/torch/cunn/pull/143#issuecomment-142771794","issue_url":"https://api.github.com/repos/torch/cunn/issues/143","id":142771794,"user":{"login":"nicolasvasilache","id":10148468,"avatar_url":"https://avatars.githubusercontent.com/u/10148468?v=3","gravatar_id":"","url":"https://api.github.com/users/nicolasvasilache","html_url":"https://github.com/nicolasvasilache","followers_url":"https://api.github.com/users/nicolasvasilache/followers","following_url":"https://api.github.com/users/nicolasvasilache/following{/other_user}","gists_url":"https://api.github.com/users/nicolasvasilache/gists{/gist_id}","starred_url":"https://api.github.com/users/nicolasvasilache/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nicolasvasilache/subscriptions","organizations_url":"https://api.github.com/users/nicolasvasilache/orgs","repos_url":"https://api.github.com/users/nicolasvasilache/repos","events_url":"https://api.github.com/users/nicolasvasilache/events{/privacy}","received_events_url":"https://api.github.com/users/nicolasvasilache/received_events","type":"User","site_admin":false},"created_at":"2015-09-24T00:58:32Z","updated_at":"2015-09-24T00:58:32Z","body":"Thanks but this is not yet optimized btw ;)\nWe could get 2-3x better perf if we wanted to spend a bit more time.\n\nOn Wed, Sep 23, 2015 at 8:28 PM, Sergey Zagoruyko <notifications@github.com>\nwrote:\n\n> An amazing work of @nicolasvasilache <https://github.com/nicolasvasilache>\n> , fast and no extra memory.\n> In fact I ported it more than a week ago and couldn't find time to factor\n> out c++11 and sm_35. So to be added:\n>\n>    - remove c++11\n>    - put a condition on sm_35 or find a workaround\n>    - port test against CPU version CPU patch and BatchNormalization are\n>    coming too.\n>\n> ------------------------------\n> You can view, comment on, or merge this pull request online at:\n>\n>   https://github.com/torch/cunn/pull/143\n> Commit Summary\n>\n>    - fast batchnorm\n>    - fixes\n>\n> File Changes\n>\n>    - *M* CMakeLists.txt\n>    <https://github.com/torch/cunn/pull/143/files#diff-0> (2)\n>    - *A* SpatialBatchNormalization.cu\n>    <https://github.com/torch/cunn/pull/143/files#diff-1> (839)\n>    - *M* init.cu <https://github.com/torch/cunn/pull/143/files#diff-2>\n>    (1)\n>    - *M* utils.h <https://github.com/torch/cunn/pull/143/files#diff-3>\n>    (1)\n>\n> Patch Links:\n>\n>    - https://github.com/torch/cunn/pull/143.patch\n>    - https://github.com/torch/cunn/pull/143.diff\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/torch/cunn/pull/143>.\n>\n"}}