{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/11569","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/11569/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/11569/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/11569/events","html_url":"https://github.com/tensorflow/tensorflow/issues/11569","id":243641745,"number":11569,"title":"Why use memory_layer in all cases?","user":{"login":"Songweiping","id":11703156,"avatar_url":"https://avatars2.githubusercontent.com/u/11703156?v=4","gravatar_id":"","url":"https://api.github.com/users/Songweiping","html_url":"https://github.com/Songweiping","followers_url":"https://api.github.com/users/Songweiping/followers","following_url":"https://api.github.com/users/Songweiping/following{/other_user}","gists_url":"https://api.github.com/users/Songweiping/gists{/gist_id}","starred_url":"https://api.github.com/users/Songweiping/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Songweiping/subscriptions","organizations_url":"https://api.github.com/users/Songweiping/orgs","repos_url":"https://api.github.com/users/Songweiping/repos","events_url":"https://api.github.com/users/Songweiping/events{/privacy}","received_events_url":"https://api.github.com/users/Songweiping/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"ebrevdo","id":1794715,"avatar_url":"https://avatars0.githubusercontent.com/u/1794715?v=4","gravatar_id":"","url":"https://api.github.com/users/ebrevdo","html_url":"https://github.com/ebrevdo","followers_url":"https://api.github.com/users/ebrevdo/followers","following_url":"https://api.github.com/users/ebrevdo/following{/other_user}","gists_url":"https://api.github.com/users/ebrevdo/gists{/gist_id}","starred_url":"https://api.github.com/users/ebrevdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ebrevdo/subscriptions","organizations_url":"https://api.github.com/users/ebrevdo/orgs","repos_url":"https://api.github.com/users/ebrevdo/repos","events_url":"https://api.github.com/users/ebrevdo/events{/privacy}","received_events_url":"https://api.github.com/users/ebrevdo/received_events","type":"User","site_admin":false},"assignees":[{"login":"ebrevdo","id":1794715,"avatar_url":"https://avatars0.githubusercontent.com/u/1794715?v=4","gravatar_id":"","url":"https://api.github.com/users/ebrevdo","html_url":"https://github.com/ebrevdo","followers_url":"https://api.github.com/users/ebrevdo/followers","following_url":"https://api.github.com/users/ebrevdo/following{/other_user}","gists_url":"https://api.github.com/users/ebrevdo/gists{/gist_id}","starred_url":"https://api.github.com/users/ebrevdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ebrevdo/subscriptions","organizations_url":"https://api.github.com/users/ebrevdo/orgs","repos_url":"https://api.github.com/users/ebrevdo/repos","events_url":"https://api.github.com/users/ebrevdo/events{/privacy}","received_events_url":"https://api.github.com/users/ebrevdo/received_events","type":"User","site_admin":false}],"milestone":null,"comments":7,"created_at":"2017-07-18T08:42:28Z","updated_at":"2018-04-09T12:37:49Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"\r\n### System information\r\n- **Have I written custom code (as opposed to using a stock example script provided in TensorFlow)**: No\r\n- **OS Platform and Distribution (e.g., Linux Ubuntu 16.04)**: ubuntu 14.04\r\n- **TensorFlow installed from (source or binary)**: binary\r\n- **TensorFlow version (use command below)**: 1.2\r\n\r\nIn current AttentionWrapper, both [BahdanauAttention](https://github.com/tensorflow/tensorflow/blob/c996c7b381a8eb54f9c7d7b298b24b1715645b68/tensorflow/contrib/seq2seq/python/ops/attention_wrapper.py#L416-L417) and [LuongAttention](https://github.com/tensorflow/tensorflow/blob/c996c7b381a8eb54f9c7d7b298b24b1715645b68/tensorflow/contrib/seq2seq/python/ops/attention_wrapper.py#L295-L296) enforce to use a memory_layer. According to my understanding, it is needed only when the depth of memory is not matched with that of query_layer. Is it intended to be in this manner?\r\n\r\n@ebrevdo , would you mind having a look at this?"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/379736925","html_url":"https://github.com/tensorflow/tensorflow/issues/11569#issuecomment-379736925","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/11569","id":379736925,"user":{"login":"tensorflowbutler","id":28546240,"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-04-09T12:37:49Z","updated_at":"2018-04-09T12:37:49Z","author_association":"MEMBER","body":"Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly."}}
{"actor":{"display_login":"tensorflowbutler"}}