{"action":"created","issue":{"url":"https://api.github.com/repos/aymericdamien/TensorFlow-Examples/issues/25","repository_url":"https://api.github.com/repos/aymericdamien/TensorFlow-Examples","labels_url":"https://api.github.com/repos/aymericdamien/TensorFlow-Examples/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/aymericdamien/TensorFlow-Examples/issues/25/comments","events_url":"https://api.github.com/repos/aymericdamien/TensorFlow-Examples/issues/25/events","html_url":"https://github.com/aymericdamien/TensorFlow-Examples/issues/25","id":151520750,"number":25,"title":"bidirectional RNN states never updated? ","user":{"login":"nrf171","id":9985138,"avatar_url":"https://avatars.githubusercontent.com/u/9985138?v=3","gravatar_id":"","url":"https://api.github.com/users/nrf171","html_url":"https://github.com/nrf171","followers_url":"https://api.github.com/users/nrf171/followers","following_url":"https://api.github.com/users/nrf171/following{/other_user}","gists_url":"https://api.github.com/users/nrf171/gists{/gist_id}","starred_url":"https://api.github.com/users/nrf171/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nrf171/subscriptions","organizations_url":"https://api.github.com/users/nrf171/orgs","repos_url":"https://api.github.com/users/nrf171/repos","events_url":"https://api.github.com/users/nrf171/events{/privacy}","received_events_url":"https://api.github.com/users/nrf171/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-04-28T00:12:10Z","updated_at":"2016-08-04T02:05:16Z","closed_at":null,"body":"Hopefully this is not stupid. In the Tensorflow tutorial, the states of the RNN are updated after each time-step and used as the states for the next time-step\r\n\r\noutput, state = lstm(current_batch_of_words, state)\r\n\r\nIn yours the state of the bidirectional RNN seems to just be discarded. Is this the case, or am I missing something? If so, why?\r\n\r\nAlso, a small detail; you use outputs[-1] for\r\n\r\nreturn tf.matmul(outputs[-1], _weights['out']) + _biases['out']\r\n\r\nwhich corresponds according to the Tensorflow API \r\n\r\n(outputs, output_state_fw, output_state_bw) = outputs\r\n\r\nSo shouldn't this be \r\n\r\nreturn tf.matmul(outputs[0], _weights['out']) + _biases['out']\r\n\r\nInstead?"},"comment":{"url":"https://api.github.com/repos/aymericdamien/TensorFlow-Examples/issues/comments/237432002","html_url":"https://github.com/aymericdamien/TensorFlow-Examples/issues/25#issuecomment-237432002","issue_url":"https://api.github.com/repos/aymericdamien/TensorFlow-Examples/issues/25","id":237432002,"user":{"login":"aymericdamien","id":10386605,"avatar_url":"https://avatars.githubusercontent.com/u/10386605?v=3","gravatar_id":"","url":"https://api.github.com/users/aymericdamien","html_url":"https://github.com/aymericdamien","followers_url":"https://api.github.com/users/aymericdamien/followers","following_url":"https://api.github.com/users/aymericdamien/following{/other_user}","gists_url":"https://api.github.com/users/aymericdamien/gists{/gist_id}","starred_url":"https://api.github.com/users/aymericdamien/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aymericdamien/subscriptions","organizations_url":"https://api.github.com/users/aymericdamien/orgs","repos_url":"https://api.github.com/users/aymericdamien/repos","events_url":"https://api.github.com/users/aymericdamien/events{/privacy}","received_events_url":"https://api.github.com/users/aymericdamien/received_events","type":"User","site_admin":false},"created_at":"2016-08-04T02:05:16Z","updated_at":"2016-08-04T02:05:16Z","body":"it has been fix"}}
{"actor":{"display_login":"aymericdamien"}}