{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/21276","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/21276/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/21276/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/21276/events","html_url":"https://github.com/tensorflow/tensorflow/pull/21276","id":346185261,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA1MTEwODEw","number":21276,"title":"Add WeightNorm wrapper layer","user":{"login":"seanpmorgan","id":18154355,"node_id":"MDQ6VXNlcjE4MTU0MzU1","avatar_url":"https://avatars3.githubusercontent.com/u/18154355?v=4","gravatar_id":"","url":"https://api.github.com/users/seanpmorgan","html_url":"https://github.com/seanpmorgan","followers_url":"https://api.github.com/users/seanpmorgan/followers","following_url":"https://api.github.com/users/seanpmorgan/following{/other_user}","gists_url":"https://api.github.com/users/seanpmorgan/gists{/gist_id}","starred_url":"https://api.github.com/users/seanpmorgan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seanpmorgan/subscriptions","organizations_url":"https://api.github.com/users/seanpmorgan/orgs","repos_url":"https://api.github.com/users/seanpmorgan/repos","events_url":"https://api.github.com/users/seanpmorgan/events{/privacy}","received_events_url":"https://api.github.com/users/seanpmorgan/received_events","type":"User","site_admin":false},"labels":[{"id":300136587,"node_id":"MDU6TGFiZWwzMDAxMzY1ODc=","url":"https://api.github.com/repos/tensorflow/tensorflow/labels/cla:%20yes","name":"cla: yes","color":"009800","default":false}],"state":"open","locked":false,"assignee":{"login":"case540","id":1299636,"node_id":"MDQ6VXNlcjEyOTk2MzY=","avatar_url":"https://avatars2.githubusercontent.com/u/1299636?v=4","gravatar_id":"","url":"https://api.github.com/users/case540","html_url":"https://github.com/case540","followers_url":"https://api.github.com/users/case540/followers","following_url":"https://api.github.com/users/case540/following{/other_user}","gists_url":"https://api.github.com/users/case540/gists{/gist_id}","starred_url":"https://api.github.com/users/case540/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/case540/subscriptions","organizations_url":"https://api.github.com/users/case540/orgs","repos_url":"https://api.github.com/users/case540/repos","events_url":"https://api.github.com/users/case540/events{/privacy}","received_events_url":"https://api.github.com/users/case540/received_events","type":"User","site_admin":false},"assignees":[{"login":"case540","id":1299636,"node_id":"MDQ6VXNlcjEyOTk2MzY=","avatar_url":"https://avatars2.githubusercontent.com/u/1299636?v=4","gravatar_id":"","url":"https://api.github.com/users/case540","html_url":"https://github.com/case540","followers_url":"https://api.github.com/users/case540/followers","following_url":"https://api.github.com/users/case540/following{/other_user}","gists_url":"https://api.github.com/users/case540/gists{/gist_id}","starred_url":"https://api.github.com/users/case540/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/case540/subscriptions","organizations_url":"https://api.github.com/users/case540/orgs","repos_url":"https://api.github.com/users/case540/repos","events_url":"https://api.github.com/users/case540/events{/privacy}","received_events_url":"https://api.github.com/users/case540/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2018-07-31T13:22:36Z","updated_at":"2018-11-21T18:55:40Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/tensorflow/tensorflow/pulls/21276","html_url":"https://github.com/tensorflow/tensorflow/pull/21276","diff_url":"https://github.com/tensorflow/tensorflow/pull/21276.diff","patch_url":"https://github.com/tensorflow/tensorflow/pull/21276.patch"},"body":"Adds a wrapper for Weight Normalization as requested in #14070 & #10125 .\r\nContains optional data dependent initialization for eager execution, and works on both keras.layers and tf.layers\r\n\r\nI struggled to figure out where to place this Wrapper, as no other layers in contrib appear to subclass anything from tf.Keras, but going forward I believe this is the direction TF is headed.  Please advise if it should go somewhere else (maybe a wrappers module in contrib?)\r\n\r\nCollab Example:\r\nhttps://colab.research.google.com/drive/1nBQSAA78oUBmi9fhnHJ_zWhHq2NXjwIc#scrollTo=au25bSP75hdr\r\n\r\nWrapped graph:\r\n![selection_002](https://user-images.githubusercontent.com/18154355/43461842-7794a942-94a2-11e8-9d28-21955273c317.png)"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/440774274","html_url":"https://github.com/tensorflow/tensorflow/pull/21276#issuecomment-440774274","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/21276","id":440774274,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0MDc3NDI3NA==","user":{"login":"tensorflowbutler","id":28546240,"node_id":"MDQ6VXNlcjI4NTQ2MjQw","avatar_url":"https://avatars2.githubusercontent.com/u/28546240?v=4","gravatar_id":"","url":"https://api.github.com/users/tensorflowbutler","html_url":"https://github.com/tensorflowbutler","followers_url":"https://api.github.com/users/tensorflowbutler/followers","following_url":"https://api.github.com/users/tensorflowbutler/following{/other_user}","gists_url":"https://api.github.com/users/tensorflowbutler/gists{/gist_id}","starred_url":"https://api.github.com/users/tensorflowbutler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tensorflowbutler/subscriptions","organizations_url":"https://api.github.com/users/tensorflowbutler/orgs","repos_url":"https://api.github.com/users/tensorflowbutler/repos","events_url":"https://api.github.com/users/tensorflowbutler/events{/privacy}","received_events_url":"https://api.github.com/users/tensorflowbutler/received_events","type":"User","site_admin":false},"created_at":"2018-11-21T18:55:40Z","updated_at":"2018-11-21T18:55:40Z","author_association":"MEMBER","body":"Nagging Assignee @case540: It has been 90 days with no activity and this issue has an assignee. Please update the label and/or status accordingly."}}
{"actor":{"display_login":"tensorflowbutler"}}