{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5827","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5827/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5827/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5827/events","html_url":"https://github.com/tensorflow/tensorflow/issues/5827","id":191474086,"number":5827,"title":"Should variables in ExponentialMovingAverage use \"get_variable\"","user":{"login":"passerbyd","id":12813325,"avatar_url":"https://avatars.githubusercontent.com/u/12813325?v=3","gravatar_id":"","url":"https://api.github.com/users/passerbyd","html_url":"https://github.com/passerbyd","followers_url":"https://api.github.com/users/passerbyd/followers","following_url":"https://api.github.com/users/passerbyd/following{/other_user}","gists_url":"https://api.github.com/users/passerbyd/gists{/gist_id}","starred_url":"https://api.github.com/users/passerbyd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/passerbyd/subscriptions","organizations_url":"https://api.github.com/users/passerbyd/orgs","repos_url":"https://api.github.com/users/passerbyd/repos","events_url":"https://api.github.com/users/passerbyd/events{/privacy}","received_events_url":"https://api.github.com/users/passerbyd/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"joel-shor","id":6020988,"avatar_url":"https://avatars.githubusercontent.com/u/6020988?v=3","gravatar_id":"","url":"https://api.github.com/users/joel-shor","html_url":"https://github.com/joel-shor","followers_url":"https://api.github.com/users/joel-shor/followers","following_url":"https://api.github.com/users/joel-shor/following{/other_user}","gists_url":"https://api.github.com/users/joel-shor/gists{/gist_id}","starred_url":"https://api.github.com/users/joel-shor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joel-shor/subscriptions","organizations_url":"https://api.github.com/users/joel-shor/orgs","repos_url":"https://api.github.com/users/joel-shor/repos","events_url":"https://api.github.com/users/joel-shor/events{/privacy}","received_events_url":"https://api.github.com/users/joel-shor/received_events","type":"User","site_admin":false},"assignees":[{"login":"joel-shor","id":6020988,"avatar_url":"https://avatars.githubusercontent.com/u/6020988?v=3","gravatar_id":"","url":"https://api.github.com/users/joel-shor","html_url":"https://github.com/joel-shor","followers_url":"https://api.github.com/users/joel-shor/followers","following_url":"https://api.github.com/users/joel-shor/following{/other_user}","gists_url":"https://api.github.com/users/joel-shor/gists{/gist_id}","starred_url":"https://api.github.com/users/joel-shor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joel-shor/subscriptions","organizations_url":"https://api.github.com/users/joel-shor/orgs","repos_url":"https://api.github.com/users/joel-shor/repos","events_url":"https://api.github.com/users/joel-shor/events{/privacy}","received_events_url":"https://api.github.com/users/joel-shor/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2016-11-24T09:44:20Z","updated_at":"2016-12-05T16:31:46Z","closed_at":"2016-12-05T16:31:46Z","body":"zero_debias for moving average was introduced recently on master branches.  (tensorflow/tensorflow/python/training/moving_averages.py)\r\nVariables in func “_zero_debias” is created by \"variable_scope.get_variable\", while slot_creator creates moving variable with \"variables.Variable\". This causes some problem when outer scope set “reuse=True”, if we only want to reuse variables of the network but not the moving averages.\r\nAt least, the new feature introduces inconsistency on how to maintain moving variables."},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/264902249","html_url":"https://github.com/tensorflow/tensorflow/issues/5827#issuecomment-264902249","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5827","id":264902249,"user":{"login":"joel-shor","id":6020988,"avatar_url":"https://avatars.githubusercontent.com/u/6020988?v=3","gravatar_id":"","url":"https://api.github.com/users/joel-shor","html_url":"https://github.com/joel-shor","followers_url":"https://api.github.com/users/joel-shor/followers","following_url":"https://api.github.com/users/joel-shor/following{/other_user}","gists_url":"https://api.github.com/users/joel-shor/gists{/gist_id}","starred_url":"https://api.github.com/users/joel-shor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joel-shor/subscriptions","organizations_url":"https://api.github.com/users/joel-shor/orgs","repos_url":"https://api.github.com/users/joel-shor/repos","events_url":"https://api.github.com/users/joel-shor/events{/privacy}","received_events_url":"https://api.github.com/users/joel-shor/received_events","type":"User","site_admin":false},"created_at":"2016-12-05T16:31:46Z","updated_at":"2016-12-05T16:31:46Z","body":"1) The issue should be fixed currently; zero_debias is no longer the default in ExponentialMovingAverage.\r\n2) There is a strong possibility that is *will* be the default in the future.\r\n3) ExponentialMovingAverage creates variables, so for future safety you might consider using it as if it respected variable scopes (ie don't put it in resuse=True variable scopes that it doesn't need to be in)"}}
{"actor":{"display_login":"joel-shor"}}