{"action":"created","issue":{"url":"https://api.github.com/repos/Theano/Theano/issues/950","labels_url":"https://api.github.com/repos/Theano/Theano/issues/950/labels{/name}","comments_url":"https://api.github.com/repos/Theano/Theano/issues/950/comments","events_url":"https://api.github.com/repos/Theano/Theano/issues/950/events","html_url":"https://github.com/Theano/Theano/issues/950","id":7102271,"number":950,"title":"Refactor Flatten, missing opt?","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/Optimization","name":"Optimization","color":"ededed"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/Theano/Theano/milestones/5","html_url":"https://github.com/Theano/Theano/milestones/0.7.1","labels_url":"https://api.github.com/repos/Theano/Theano/milestones/5/labels","id":1034905,"number":5,"title":"0.7.1","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":31,"closed_issues":10,"state":"open","created_at":"2015-03-23T18:36:30Z","updated_at":"2015-12-14T18:14:08Z","due_on":null,"closed_at":null},"comments":7,"created_at":"2012-09-24T19:03:45Z","updated_at":"2015-12-14T18:14:08Z","closed_at":"2015-12-14T18:14:08Z","body":"We have the Flatten op. Why do we need it? We could reuse the reshape op. Having less op make the optimization easier.\r\n\r\nWe can reuse reshape(-1), this won't introduce bigger or uglier graph\r\n\r\nTODO:\r\n- [ ] Comment the Flatten op and use reshape(-1) where needed.\r\n- [ ] Check optimization that would check for flatten, update them to check for the old Flatten and the new pattern. Maybe refactor this into a method?\r\n- [ ] The user flatten interface should stay and should introduce reshape(-1)\r\n- [ ] Then uncomment the Flatten op, change its description to tell it is deprecated (needed to unpickle old graph)."},"comment":{"url":"https://api.github.com/repos/Theano/Theano/issues/comments/164514664","html_url":"https://github.com/Theano/Theano/issues/950#issuecomment-164514664","issue_url":"https://api.github.com/repos/Theano/Theano/issues/950","id":164514664,"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":"2015-12-14T18:14:08Z","updated_at":"2015-12-14T18:14:08Z","body":"I believe this is closed by #3480, let's re-open it if something is missing."}}