{"action":"created","issue":{"url":"https://api.github.com/repos/lisa-lab/pylearn2/issues/1354","labels_url":"https://api.github.com/repos/lisa-lab/pylearn2/issues/1354/labels{/name}","comments_url":"https://api.github.com/repos/lisa-lab/pylearn2/issues/1354/comments","events_url":"https://api.github.com/repos/lisa-lab/pylearn2/issues/1354/events","html_url":"https://github.com/lisa-lab/pylearn2/issues/1354","id":54688769,"number":1354,"title":"compute_test_err.py is broken for mnist.yaml in maxout","user":{"login":"JesseLivezey","id":2081702,"avatar_url":"https://avatars.githubusercontent.com/u/2081702?v=3","gravatar_id":"","url":"https://api.github.com/users/JesseLivezey","html_url":"https://github.com/JesseLivezey","followers_url":"https://api.github.com/users/JesseLivezey/followers","following_url":"https://api.github.com/users/JesseLivezey/following{/other_user}","gists_url":"https://api.github.com/users/JesseLivezey/gists{/gist_id}","starred_url":"https://api.github.com/users/JesseLivezey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JesseLivezey/subscriptions","organizations_url":"https://api.github.com/users/JesseLivezey/orgs","repos_url":"https://api.github.com/users/JesseLivezey/repos","events_url":"https://api.github.com/users/JesseLivezey/events{/privacy}","received_events_url":"https://api.github.com/users/JesseLivezey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-18T07:06:50Z","updated_at":"2015-01-19T23:32:09Z","closed_at":null,"body":"This line:\r\nhttps://github.com/lisa-lab/pylearn2/blob/master/pylearn2/scripts/papers/maxout/compute_test_err.py#L55\r\nwill fail because x_arg will already be in a Conv2D space for mnist.yaml.\r\n\r\nSee discussion here:\r\nhttps://groups.google.com/forum/#!topic/pylearn-users/WAo8x3LzvdU"},"comment":{"url":"https://api.github.com/repos/lisa-lab/pylearn2/issues/comments/70579685","html_url":"https://github.com/lisa-lab/pylearn2/issues/1354#issuecomment-70579685","issue_url":"https://api.github.com/repos/lisa-lab/pylearn2/issues/1354","id":70579685,"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-01-19T23:32:09Z","updated_at":"2015-01-19T23:32:09Z","body":"That's correct, lines 54 and 55 should be removed, because the iterator is already doing the conversion."}}