{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/1868","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/1868/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/1868/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/1868/events","html_url":"https://github.com/tensorflow/tensorflow/pull/1868","id":147639787,"number":1868,"title":"Fix conv2d with kernel 1x1 stride > 1","user":{"login":"ry","id":80,"avatar_url":"https://avatars.githubusercontent.com/u/80?v=3","gravatar_id":"","url":"https://api.github.com/users/ry","html_url":"https://github.com/ry","followers_url":"https://api.github.com/users/ry/followers","following_url":"https://api.github.com/users/ry/following{/other_user}","gists_url":"https://api.github.com/users/ry/gists{/gist_id}","starred_url":"https://api.github.com/users/ry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ry/subscriptions","organizations_url":"https://api.github.com/users/ry/orgs","repos_url":"https://api.github.com/users/ry/repos","events_url":"https://api.github.com/users/ry/events{/privacy}","received_events_url":"https://api.github.com/users/ry/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/tensorflow/tensorflow/labels/cla:%20yes","name":"cla: yes","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-04-12T05:48:58Z","updated_at":"2016-04-12T06:00:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tensorflow/tensorflow/pulls/1868","html_url":"https://github.com/tensorflow/tensorflow/pull/1868","diff_url":"https://github.com/tensorflow/tensorflow/pull/1868.diff","patch_url":"https://github.com/tensorflow/tensorflow/pull/1868.patch"},"body":"Fixes #889 - seems the problem was an optimization to call directly into cublas when the kernel is 1x1. I suspect that was meant for 1x1 and stride 1, as it calculates the wrong thing with stride1. This adds 3 tests - two that demo the bug, one that I did while debugging but figured to submit too for extra coverage.\r\n\r\nI signed the CLA"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/208721381","html_url":"https://github.com/tensorflow/tensorflow/pull/1868#issuecomment-208721381","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/1868","id":208721381,"user":{"login":"vrv","id":463737,"avatar_url":"https://avatars.githubusercontent.com/u/463737?v=3","gravatar_id":"","url":"https://api.github.com/users/vrv","html_url":"https://github.com/vrv","followers_url":"https://api.github.com/users/vrv/followers","following_url":"https://api.github.com/users/vrv/following{/other_user}","gists_url":"https://api.github.com/users/vrv/gists{/gist_id}","starred_url":"https://api.github.com/users/vrv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vrv/subscriptions","organizations_url":"https://api.github.com/users/vrv/orgs","repos_url":"https://api.github.com/users/vrv/repos","events_url":"https://api.github.com/users/vrv/events{/privacy}","received_events_url":"https://api.github.com/users/vrv/received_events","type":"User","site_admin":false},"created_at":"2016-04-12T06:00:55Z","updated_at":"2016-04-12T06:00:55Z","body":"Very nice, and good catch there!  Just a few small comments, and then we can test / merge."}}