{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/3460","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/3460/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/3460/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/3460/events","html_url":"https://github.com/tensorflow/tensorflow/issues/3460","id":167004495,"number":3460,"title":"Exponential Moving Average of trained variables in tf.contrib.learn.estimators.estimator","user":{"login":"marcoadurno","id":4450528,"avatar_url":"https://avatars.githubusercontent.com/u/4450528?v=3","gravatar_id":"","url":"https://api.github.com/users/marcoadurno","html_url":"https://github.com/marcoadurno","followers_url":"https://api.github.com/users/marcoadurno/followers","following_url":"https://api.github.com/users/marcoadurno/following{/other_user}","gists_url":"https://api.github.com/users/marcoadurno/gists{/gist_id}","starred_url":"https://api.github.com/users/marcoadurno/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marcoadurno/subscriptions","organizations_url":"https://api.github.com/users/marcoadurno/orgs","repos_url":"https://api.github.com/users/marcoadurno/repos","events_url":"https://api.github.com/users/marcoadurno/events{/privacy}","received_events_url":"https://api.github.com/users/marcoadurno/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"martinwicke","id":577277,"avatar_url":"https://avatars.githubusercontent.com/u/577277?v=3","gravatar_id":"","url":"https://api.github.com/users/martinwicke","html_url":"https://github.com/martinwicke","followers_url":"https://api.github.com/users/martinwicke/followers","following_url":"https://api.github.com/users/martinwicke/following{/other_user}","gists_url":"https://api.github.com/users/martinwicke/gists{/gist_id}","starred_url":"https://api.github.com/users/martinwicke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinwicke/subscriptions","organizations_url":"https://api.github.com/users/martinwicke/orgs","repos_url":"https://api.github.com/users/martinwicke/repos","events_url":"https://api.github.com/users/martinwicke/events{/privacy}","received_events_url":"https://api.github.com/users/martinwicke/received_events","type":"User","site_admin":false},"assignees":[{"login":"martinwicke","id":577277,"avatar_url":"https://avatars.githubusercontent.com/u/577277?v=3","gravatar_id":"","url":"https://api.github.com/users/martinwicke","html_url":"https://github.com/martinwicke","followers_url":"https://api.github.com/users/martinwicke/followers","following_url":"https://api.github.com/users/martinwicke/following{/other_user}","gists_url":"https://api.github.com/users/martinwicke/gists{/gist_id}","starred_url":"https://api.github.com/users/martinwicke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinwicke/subscriptions","organizations_url":"https://api.github.com/users/martinwicke/orgs","repos_url":"https://api.github.com/users/martinwicke/repos","events_url":"https://api.github.com/users/martinwicke/events{/privacy}","received_events_url":"https://api.github.com/users/martinwicke/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2016-07-22T09:20:33Z","updated_at":"2016-08-24T21:00:56Z","closed_at":null,"body":"Hi,\r\n\r\nAs mentioned in the cifar10 tutorial, it is often useful to use an exponential moving average of the trained variables at prediction time.\r\n\r\nFrom my understanding, in it's current form, the class BaseEstimator does not allow to do so. \r\n\r\nThe _infer_model method of BaseEstimator would need to create a saver object that restores the averaged version of the trained variables, and the saver object would then need to be cascaded into graph_actions.infer and ultimately down to graph_actions.run_feeds.\r\n\r\nWhilst overriding a class method is not a big deal, I was wondering if there was a neat way of achieving the end goal without having to refactor the graph_actions.\r\n\r\nBest Regards,\r\n\r\nMarco"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/242206589","html_url":"https://github.com/tensorflow/tensorflow/issues/3460#issuecomment-242206589","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/3460","id":242206589,"user":{"login":"martinwicke","id":577277,"avatar_url":"https://avatars.githubusercontent.com/u/577277?v=3","gravatar_id":"","url":"https://api.github.com/users/martinwicke","html_url":"https://github.com/martinwicke","followers_url":"https://api.github.com/users/martinwicke/followers","following_url":"https://api.github.com/users/martinwicke/following{/other_user}","gists_url":"https://api.github.com/users/martinwicke/gists{/gist_id}","starred_url":"https://api.github.com/users/martinwicke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinwicke/subscriptions","organizations_url":"https://api.github.com/users/martinwicke/orgs","repos_url":"https://api.github.com/users/martinwicke/repos","events_url":"https://api.github.com/users/martinwicke/events{/privacy}","received_events_url":"https://api.github.com/users/martinwicke/received_events","type":"User","site_admin":false},"created_at":"2016-08-24T21:00:56Z","updated_at":"2016-08-24T21:00:56Z","body":"I think if you simply add the moving averages to the graph and explicitly\nadd their output to the VARIABLES collection they will be saved and you can\nuse them in the inference graph.\n\nThis is assuming you use the Estimator class. There should be no need to\noverride anything.\nOn Mon, Aug 15, 2016 at 16:37 Daniel W Mane <notifications@github.com>\nwrote:\n\n> @martinwicke <https://github.com/martinwicke> Can you comment please?\n>\n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/tensorflow/tensorflow/issues/3460#issuecomment-239960552>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/AAjO_XL8t3C19ix8K12DAO0Uz5Ip8oOUks5qgPgbgaJpZM4JSlat>\n> .\n>\n"}}
{"actor":{"display_login":"martinwicke"}}