{"action":"created","issue":{"url":"https://api.github.com/repos/Theano/Theano/issues/2599","labels_url":"https://api.github.com/repos/Theano/Theano/issues/2599/labels{/name}","comments_url":"https://api.github.com/repos/Theano/Theano/issues/2599/comments","events_url":"https://api.github.com/repos/Theano/Theano/issues/2599/events","html_url":"https://github.com/Theano/Theano/pull/2599","id":60751636,"number":2599,"title":"fixed error due to integer border_mode in dnn.py","user":{"login":"gwding","id":7264698,"avatar_url":"https://avatars.githubusercontent.com/u/7264698?v=3","gravatar_id":"","url":"https://api.github.com/users/gwding","html_url":"https://github.com/gwding","followers_url":"https://api.github.com/users/gwding/followers","following_url":"https://api.github.com/users/gwding/following{/other_user}","gists_url":"https://api.github.com/users/gwding/gists{/gist_id}","starred_url":"https://api.github.com/users/gwding/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gwding/subscriptions","organizations_url":"https://api.github.com/users/gwding/orgs","repos_url":"https://api.github.com/users/gwding/repos","events_url":"https://api.github.com/users/gwding/events{/privacy}","received_events_url":"https://api.github.com/users/gwding/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-12T00:13:58Z","updated_at":"2015-03-12T00:17:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Theano/Theano/pulls/2599","html_url":"https://github.com/Theano/Theano/pull/2599","diff_url":"https://github.com/Theano/Theano/pull/2599.diff","patch_url":"https://github.com/Theano/Theano/pull/2599.patch"},"body":"the situation of border_mode as int was not considered in the function get_out_shape in dnn.py, in the latest versions. "},"comment":{"url":"https://api.github.com/repos/Theano/Theano/issues/comments/78401288","html_url":"https://github.com/Theano/Theano/pull/2599#issuecomment-78401288","issue_url":"https://api.github.com/repos/Theano/Theano/issues/2599","id":78401288,"user":{"login":"nouiz","id":180987,"avatar_url":"https://avatars.githubusercontent.com/u/180987?v=3","gravatar_id":"","url":"https://api.github.com/users/nouiz","html_url":"https://github.com/nouiz","followers_url":"https://api.github.com/users/nouiz/followers","following_url":"https://api.github.com/users/nouiz/following{/other_user}","gists_url":"https://api.github.com/users/nouiz/gists{/gist_id}","starred_url":"https://api.github.com/users/nouiz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nouiz/subscriptions","organizations_url":"https://api.github.com/users/nouiz/orgs","repos_url":"https://api.github.com/users/nouiz/repos","events_url":"https://api.github.com/users/nouiz/events{/privacy}","received_events_url":"https://api.github.com/users/nouiz/received_events","type":"User","site_admin":false},"created_at":"2015-03-12T00:17:36Z","updated_at":"2015-03-12T00:17:36Z","body":"thanks. I think the PR is good, but can you add a test? The goal is that we don't break it again later.\r\nIf you didn't have added a test tomorrow morning, I'll merge this PR to get the crash fix."}}