{"push_id":1037686999,"size":1,"distinct_size":1,"ref":"refs/heads/fix-temporal-convolution","head":"02bc67c446d0c42b7831a83ba15bff9677a372d2","before":"372a685982d31df053c4a7c4bbc1882bca4c31c9","commits":[{"sha":"02bc67c446d0c42b7831a83ba15bff9677a372d2","author":{"email":"7c55a39df3f2caf2e852dae4b7066c85e85c2a84@nvidia.com","name":"Greg Heinrich"},"message":"Fix TemporalConvolution output size\n\nIssue seen in updateOutput() when batch size is decreasing (e.g. 256->128):\n```\n.../torch/install/share/lua/5.1/torch/Tensor.lua:462: Wrong size for view. Input size: 66060288. Output size: 128x256x1008x1\nstack traceback:\n\t[C]: in function 'error'\n\t.../torch/install/share/lua/5.1/torch/Tensor.lua:462: in function 'view'\n\t...orch/install/share/lua/5.1/cudnn/TemporalConvolution.lua:62: in function <...orch/install/share/lua/5.1/cudnn/TemporalConvolution.lua:54>\n\n```\n\nSee new test cudnntest.TemporalConvolution_reduceBatchSize() for a repro.\n\nCalling a tensor's set() method without extra parameters but the requested storage causes the tensor to see a 1-D view of the full storage (not the associated tensor). For example:\n```\nth> x=torch.Tensor(10)\n                                                                      [0.0001s]\nth> y=x:resize(5)\n                                                                      [0.0001s]\nth> torch.Tensor():set(y:storage()):size()\n 10\n[torch.LongStorage of size 1]\n\n                                                                      [0.0002s]\n\n```\n\nProposed fix is to specify the desired view through the optional parameters of the set() method.","distinct":true,"url":"https://api.github.com/repos/gheinrich/cudnn.torch/commits/02bc67c446d0c42b7831a83ba15bff9677a372d2"}]}