{"action":"opened","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/6582","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/6582/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/6582/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/6582/events","html_url":"https://github.com/tensorflow/tensorflow/issues/6582","id":198235905,"number":6582,"title":"Incompatible state size between LSTMCell and LSTMBlockCell","user":{"login":"jihunchoi","id":1898501,"avatar_url":"https://avatars.githubusercontent.com/u/1898501?v=3","gravatar_id":"","url":"https://api.github.com/users/jihunchoi","html_url":"https://github.com/jihunchoi","followers_url":"https://api.github.com/users/jihunchoi/followers","following_url":"https://api.github.com/users/jihunchoi/following{/other_user}","gists_url":"https://api.github.com/users/jihunchoi/gists{/gist_id}","starred_url":"https://api.github.com/users/jihunchoi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jihunchoi/subscriptions","organizations_url":"https://api.github.com/users/jihunchoi/orgs","repos_url":"https://api.github.com/users/jihunchoi/repos","events_url":"https://api.github.com/users/jihunchoi/events{/privacy}","received_events_url":"https://api.github.com/users/jihunchoi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-12-31T15:20:40Z","updated_at":"2016-12-31T15:20:40Z","closed_at":null,"body":"Hello,\r\n\r\nI heard that the implementation of LSTMBlockCell is faster than that of LSTMCell, so I am trying to convert my codes using LSTMCell to use LSTMBlockCell.\r\nHowever, I found one weird point: the types of state_size of the two are not compatible.\r\nSpecifically, LSTMCell's state_size is of type LSTMStateTuple and LSTMBlockCell's state_size is of type tuple.\r\nAs far as I know, tuple and LSTMStateTuple are quite similar but the latter handles an internal memory state and a hidden state in a more abstract way.\r\n\r\nI know the separated use of tuple and LSTMStateTuple is quite useful when I have to detect whether a cell is MultiRNNCell (where state_size is of type tuple) or a cell is LSTMCell (where state_size is of type LSTMStateTuple).\r\nThus, I think the current behavior of LSTMBlockCell should be corrected to use LSTMStateTuple as a state type.\r\nIt can be easily modified if there is no backward compatibility issue, and I will send a pull request if nobody is working on this now."}}
{"actor":{"display_login":"jihunchoi"}}