{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/cleverhans/issues/856","repository_url":"https://api.github.com/repos/tensorflow/cleverhans","labels_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/856/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/856/comments","events_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/856/events","html_url":"https://github.com/tensorflow/cleverhans/issues/856","id":374223548,"node_id":"MDU6SXNzdWUzNzQyMjM1NDg=","number":856,"title":"Accept sess=None in attacks","user":{"login":"erfannoury","id":1921165,"node_id":"MDQ6VXNlcjE5MjExNjU=","avatar_url":"https://avatars0.githubusercontent.com/u/1921165?v=4","gravatar_id":"","url":"https://api.github.com/users/erfannoury","html_url":"https://github.com/erfannoury","followers_url":"https://api.github.com/users/erfannoury/followers","following_url":"https://api.github.com/users/erfannoury/following{/other_user}","gists_url":"https://api.github.com/users/erfannoury/gists{/gist_id}","starred_url":"https://api.github.com/users/erfannoury/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erfannoury/subscriptions","organizations_url":"https://api.github.com/users/erfannoury/orgs","repos_url":"https://api.github.com/users/erfannoury/repos","events_url":"https://api.github.com/users/erfannoury/events{/privacy}","received_events_url":"https://api.github.com/users/erfannoury/received_events","type":"User","site_admin":false},"labels":[{"id":445392043,"node_id":"MDU6TGFiZWw0NDUzOTIwNDM=","url":"https://api.github.com/repos/tensorflow/cleverhans/labels/new%20feature","name":"new feature","color":"fef2c0","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-10-26T04:36:31Z","updated_at":"2018-10-30T20:10:19Z","closed_at":null,"author_association":"NONE","body":"With the current implementation, every object created from the classes based on the `Attack` class need to provide a `tf.Session` instance in the constructor. However, this limits the usage of the attacks to cases where a `Session` has already been created. For instance, there are problems for use cases in which the attacks are used in the graph construction phase and `tf.Session` will be created later and will not be accessible to the user. This happens when using the Estimator API in which attacks can either be used for training or for evaluation. In both cases the graph will be constructed first and creating the session and executing the graph is handled by the Estimator class. Would it be reasonable to move [this check](https://github.com/tensorflow/cleverhans/blob/master/cleverhans/attacks.py#L51-L52) to where `sess` actually will be used?"},"comment":{"url":"https://api.github.com/repos/tensorflow/cleverhans/issues/comments/434451306","html_url":"https://github.com/tensorflow/cleverhans/issues/856#issuecomment-434451306","issue_url":"https://api.github.com/repos/tensorflow/cleverhans/issues/856","id":434451306,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNDQ1MTMwNg==","user":{"login":"erfannoury","id":1921165,"node_id":"MDQ6VXNlcjE5MjExNjU=","avatar_url":"https://avatars0.githubusercontent.com/u/1921165?v=4","gravatar_id":"","url":"https://api.github.com/users/erfannoury","html_url":"https://github.com/erfannoury","followers_url":"https://api.github.com/users/erfannoury/followers","following_url":"https://api.github.com/users/erfannoury/following{/other_user}","gists_url":"https://api.github.com/users/erfannoury/gists{/gist_id}","starred_url":"https://api.github.com/users/erfannoury/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erfannoury/subscriptions","organizations_url":"https://api.github.com/users/erfannoury/orgs","repos_url":"https://api.github.com/users/erfannoury/repos","events_url":"https://api.github.com/users/erfannoury/events{/privacy}","received_events_url":"https://api.github.com/users/erfannoury/received_events","type":"User","site_admin":false},"created_at":"2018-10-30T20:10:19Z","updated_at":"2018-10-30T20:10:19Z","author_association":"NONE","body":"@AlexeyKurakin Sure, I can start working on them and will submit a PR as soon as possible."}}
{"actor":{"display_login":"erfannoury"}}