{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/cleverhans/issues/480","repository_url":"https://api.github.com/repos/tensorflow/cleverhans","labels_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/480/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/480/comments","events_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/480/events","html_url":"https://github.com/tensorflow/cleverhans/pull/480","id":346061224,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA1MDE1MjAx","number":480,"title":"Moving a snippet of code to a better place","user":{"login":"aashish-kumar","id":2477787,"node_id":"MDQ6VXNlcjI0Nzc3ODc=","avatar_url":"https://avatars3.githubusercontent.com/u/2477787?v=4","gravatar_id":"","url":"https://api.github.com/users/aashish-kumar","html_url":"https://github.com/aashish-kumar","followers_url":"https://api.github.com/users/aashish-kumar/followers","following_url":"https://api.github.com/users/aashish-kumar/following{/other_user}","gists_url":"https://api.github.com/users/aashish-kumar/gists{/gist_id}","starred_url":"https://api.github.com/users/aashish-kumar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aashish-kumar/subscriptions","organizations_url":"https://api.github.com/users/aashish-kumar/orgs","repos_url":"https://api.github.com/users/aashish-kumar/repos","events_url":"https://api.github.com/users/aashish-kumar/events{/privacy}","received_events_url":"https://api.github.com/users/aashish-kumar/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-07-31T07:06:41Z","updated_at":"2018-07-31T19:34:57Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/tensorflow/cleverhans/pulls/480","html_url":"https://github.com/tensorflow/cleverhans/pull/480","diff_url":"https://github.com/tensorflow/cleverhans/pull/480.diff","patch_url":"https://github.com/tensorflow/cleverhans/pull/480.patch"},"body":"As part of creating graph, None arguments are removed. This works as expected, but when generate_np is called in a loop, graph will be constructed only during first iteration. So further calls to generate_np can contain None arguments(as they were removed in function construct_graph) which result in following code to throw error:\r\n\r\n`        for name in feedable:\r\n             feed_dict[new_kwargs[name]] = feedable[name]`\r\n\r\nNone arguments should be removed for every call to generate_np so that a single class instantiation could be used."},"comment":{"url":"https://api.github.com/repos/tensorflow/cleverhans/issues/comments/409341616","html_url":"https://github.com/tensorflow/cleverhans/pull/480#issuecomment-409341616","issue_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/480","id":409341616,"node_id":"MDEyOklzc3VlQ29tbWVudDQwOTM0MTYxNg==","user":{"login":"goodfeli","id":387866,"node_id":"MDQ6VXNlcjM4Nzg2Ng==","avatar_url":"https://avatars3.githubusercontent.com/u/387866?v=4","gravatar_id":"","url":"https://api.github.com/users/goodfeli","html_url":"https://github.com/goodfeli","followers_url":"https://api.github.com/users/goodfeli/followers","following_url":"https://api.github.com/users/goodfeli/following{/other_user}","gists_url":"https://api.github.com/users/goodfeli/gists{/gist_id}","starred_url":"https://api.github.com/users/goodfeli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/goodfeli/subscriptions","organizations_url":"https://api.github.com/users/goodfeli/orgs","repos_url":"https://api.github.com/users/goodfeli/repos","events_url":"https://api.github.com/users/goodfeli/events{/privacy}","received_events_url":"https://api.github.com/users/goodfeli/received_events","type":"User","site_admin":false},"created_at":"2018-07-31T19:34:57Z","updated_at":"2018-07-31T19:34:57Z","author_association":"COLLABORATOR","body":"The build is fixed now. I restarted the build for this PR."}}
{"actor":{"display_login":"goodfeli"}}