{"action":"created","issue":{"url":"https://api.github.com/repos/Theano/Theano/issues/2803","labels_url":"https://api.github.com/repos/Theano/Theano/issues/2803/labels{/name}","comments_url":"https://api.github.com/repos/Theano/Theano/issues/2803/comments","events_url":"https://api.github.com/repos/Theano/Theano/issues/2803/events","html_url":"https://github.com/Theano/Theano/pull/2803","id":69860255,"number":2803,"title":"[CRASH] Scan segfault with inconsistent inner graphs","user":{"login":"carriepl","id":832811,"avatar_url":"https://avatars.githubusercontent.com/u/832811?v=3","gravatar_id":"","url":"https://api.github.com/users/carriepl","html_url":"https://github.com/carriepl","followers_url":"https://api.github.com/users/carriepl/followers","following_url":"https://api.github.com/users/carriepl/following{/other_user}","gists_url":"https://api.github.com/users/carriepl/gists{/gist_id}","starred_url":"https://api.github.com/users/carriepl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/carriepl/subscriptions","organizations_url":"https://api.github.com/users/carriepl/orgs","repos_url":"https://api.github.com/users/carriepl/repos","events_url":"https://api.github.com/users/carriepl/events{/privacy}","received_events_url":"https://api.github.com/users/carriepl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-04-21T14:25:10Z","updated_at":"2015-04-24T01:15:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Theano/Theano/pulls/2803","html_url":"https://github.com/Theano/Theano/pull/2803","diff_url":"https://github.com/Theano/Theano/pull/2803.diff","patch_url":"https://github.com/Theano/Theano/pull/2803.patch"},"body":"This PR adds some validation and a test case for the recent segfault issue when the inner graph has inconsistencies. The validation ensures that, for recurrent states, the inner inputs and inner outputs have the same type and that there are no GPU variables in the inputs and/or outputs is the flags in scan_op.info are not set correctly.\r\n\r\nThis also includes some refactoring of some helper functions in Scan_op to travel between inner and outer variables as well as between inputs and outputs."},"comment":{"url":"https://api.github.com/repos/Theano/Theano/issues/comments/95766602","html_url":"https://github.com/Theano/Theano/pull/2803#issuecomment-95766602","issue_url":"https://api.github.com/repos/Theano/Theano/issues/2803","id":95766602,"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-04-24T01:15:17Z","updated_at":"2015-04-24T01:15:17Z","body":"So it is good for me, bit someone else should double check.\nLe 23 avr. 2015 15:30, \"carriepl\" <notifications@github.com> a écrit :\n\n> It works with 1.8.1. Earlier versions seem to conflict with python 3.4.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/Theano/Theano/pull/2803#issuecomment-95693741>.\n>\n"}}