{"action":"created","issue":{"url":"https://api.github.com/repos/Theano/Theano/issues/4933","repository_url":"https://api.github.com/repos/Theano/Theano","labels_url":"https://api.github.com/repos/Theano/Theano/issues/4933/labels{/name}","comments_url":"https://api.github.com/repos/Theano/Theano/issues/4933/comments","events_url":"https://api.github.com/repos/Theano/Theano/issues/4933/events","html_url":"https://github.com/Theano/Theano/issues/4933","id":174822061,"number":4933,"title":"User interface change: pool_2d rename parameter ds to ws","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},"labels":[{"url":"https://api.github.com/repos/Theano/Theano/labels/CCW","name":"CCW","color":"f7c6c7"},{"url":"https://api.github.com/repos/Theano/Theano/labels/Interface","name":"Interface","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/Theano/Theano/milestones/6","html_url":"https://github.com/Theano/Theano/milestones/0.9","labels_url":"https://api.github.com/repos/Theano/Theano/milestones/6/labels","id":1586579,"number":6,"title":"0.9","description":"","creator":{"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},"open_issues":51,"closed_issues":7,"state":"open","created_at":"2016-02-16T19:03:10Z","updated_at":"2016-10-13T19:53:31Z","due_on":null,"closed_at":null},"comments":2,"created_at":"2016-09-02T18:34:01Z","updated_at":"2016-10-17T14:11:46Z","closed_at":null,"body":"Both GPU back-end use ws. Some internal stuff in the gpu back-ends is fixed by this PR:\r\n\r\nhttps://github.com/Theano/Theano/pull/4756/\r\n\r\nBut this don't change the CPU user interface, as it is using ds for historical reason (downsample size).\r\n\r\nI think we should be consistent in the user interface in CPU and GPU and I prefer ws to ds.\r\n\r\nWe must keep the support for the old interface with a deprecation warning and we should support unpickling old files and automatically convert them to the new interface."},"comment":{"url":"https://api.github.com/repos/Theano/Theano/issues/comments/254218642","html_url":"https://github.com/Theano/Theano/issues/4933#issuecomment-254218642","issue_url":"https://api.github.com/repos/Theano/Theano/issues/4933","id":254218642,"user":{"login":"lamblin","id":178249,"avatar_url":"https://avatars.githubusercontent.com/u/178249?v=3","gravatar_id":"","url":"https://api.github.com/users/lamblin","html_url":"https://github.com/lamblin","followers_url":"https://api.github.com/users/lamblin/followers","following_url":"https://api.github.com/users/lamblin/following{/other_user}","gists_url":"https://api.github.com/users/lamblin/gists{/gist_id}","starred_url":"https://api.github.com/users/lamblin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lamblin/subscriptions","organizations_url":"https://api.github.com/users/lamblin/orgs","repos_url":"https://api.github.com/users/lamblin/repos","events_url":"https://api.github.com/users/lamblin/events{/privacy}","received_events_url":"https://api.github.com/users/lamblin/received_events","type":"User","site_admin":false},"created_at":"2016-10-17T14:11:46Z","updated_at":"2016-10-17T14:11:46Z","body":"Now that `pool_3d` has been merged, the same change should be done there as well.\r\nIf it is done quickly enough, I think we don't need to worry about deprecation warnings and compatibility."}}
{"actor":{"display_login":"lamblin"}}