{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/16703","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/16703/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/16703/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/16703/events","html_url":"https://github.com/tensorflow/tensorflow/issues/16703","id":293988052,"number":16703,"title":"tf.contrib.rnn.GLSTMCell is hilariously broken","user":{"login":"akhti","id":7470747,"avatar_url":"https://avatars3.githubusercontent.com/u/7470747?v=4","gravatar_id":"","url":"https://api.github.com/users/akhti","html_url":"https://github.com/akhti","followers_url":"https://api.github.com/users/akhti/followers","following_url":"https://api.github.com/users/akhti/following{/other_user}","gists_url":"https://api.github.com/users/akhti/gists{/gist_id}","starred_url":"https://api.github.com/users/akhti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akhti/subscriptions","organizations_url":"https://api.github.com/users/akhti/orgs","repos_url":"https://api.github.com/users/akhti/repos","events_url":"https://api.github.com/users/akhti/events{/privacy}","received_events_url":"https://api.github.com/users/akhti/received_events","type":"User","site_admin":false},"labels":[{"id":404586594,"url":"https://api.github.com/repos/tensorflow/tensorflow/labels/stat:awaiting%20tensorflower","name":"stat:awaiting tensorflower","color":"b60205","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-02-02T18:54:15Z","updated_at":"2018-02-02T23:25:29Z","closed_at":null,"author_association":"NONE","body":"In 3f579020bab8f00e4621e9c7c740cbf13136a809 an \"if\" was added that caches linear transformation weights:\r\nhttps://github.com/tensorflow/tensorflow/blob/r1.5/tensorflow/contrib/rnn/python/ops/rnn_cell.py#L2316\r\n\r\nThe problem is that this _linear is inside a loop. And so the change tied weights of all these linear transformations.\r\n\r\nCC @okuchaiev \r\n"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/362739094","html_url":"https://github.com/tensorflow/tensorflow/issues/16703#issuecomment-362739094","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/16703","id":362739094,"user":{"login":"panyx0718","id":2887803,"avatar_url":"https://avatars0.githubusercontent.com/u/2887803?v=4","gravatar_id":"","url":"https://api.github.com/users/panyx0718","html_url":"https://github.com/panyx0718","followers_url":"https://api.github.com/users/panyx0718/followers","following_url":"https://api.github.com/users/panyx0718/following{/other_user}","gists_url":"https://api.github.com/users/panyx0718/gists{/gist_id}","starred_url":"https://api.github.com/users/panyx0718/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/panyx0718/subscriptions","organizations_url":"https://api.github.com/users/panyx0718/orgs","repos_url":"https://api.github.com/users/panyx0718/repos","events_url":"https://api.github.com/users/panyx0718/events{/privacy}","received_events_url":"https://api.github.com/users/panyx0718/received_events","type":"User","site_admin":false},"created_at":"2018-02-02T23:25:29Z","updated_at":"2018-02-02T23:25:29Z","author_association":"NONE","body":"Right. this is a bug introduced during a refactoring. Perhaps we should cache one Linear for each group_id."}}
{"actor":{"display_login":"panyx0718"}}