{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/738","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/738/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/738/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/738/events","html_url":"https://github.com/tensorflow/tensorflow/pull/738","id":125860238,"number":738,"title":"Speeds up the bidirectional RNN by providing the sequence length to e…","user":{"login":"kentonl","id":2646947,"avatar_url":"https://avatars.githubusercontent.com/u/2646947?v=3","gravatar_id":"","url":"https://api.github.com/users/kentonl","html_url":"https://github.com/kentonl","followers_url":"https://api.github.com/users/kentonl/followers","following_url":"https://api.github.com/users/kentonl/following{/other_user}","gists_url":"https://api.github.com/users/kentonl/gists{/gist_id}","starred_url":"https://api.github.com/users/kentonl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kentonl/subscriptions","organizations_url":"https://api.github.com/users/kentonl/orgs","repos_url":"https://api.github.com/users/kentonl/repos","events_url":"https://api.github.com/users/kentonl/events{/privacy}","received_events_url":"https://api.github.com/users/kentonl/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/tensorflow/tensorflow/labels/cla:%20yes","name":"cla: yes","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"ebrevdo","id":1794715,"avatar_url":"https://avatars.githubusercontent.com/u/1794715?v=3","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":23,"created_at":"2016-01-11T01:47:15Z","updated_at":"2016-01-12T04:50:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tensorflow/tensorflow/pulls/738","html_url":"https://github.com/tensorflow/tensorflow/pull/738","diff_url":"https://github.com/tensorflow/tensorflow/pull/738.diff","patch_url":"https://github.com/tensorflow/tensorflow/pull/738.patch"},"body":"…ach RNN. Also fixes shape inference when using the sequence length.\r\n\r\nChange-Id: Ib2843c620bc4bd348aafb2d676b61fa35f1223af"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/170790503","html_url":"https://github.com/tensorflow/tensorflow/pull/738#issuecomment-170790503","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/738","id":170790503,"user":{"login":"kentonl","id":2646947,"avatar_url":"https://avatars.githubusercontent.com/u/2646947?v=3","gravatar_id":"","url":"https://api.github.com/users/kentonl","html_url":"https://github.com/kentonl","followers_url":"https://api.github.com/users/kentonl/followers","following_url":"https://api.github.com/users/kentonl/following{/other_user}","gists_url":"https://api.github.com/users/kentonl/gists{/gist_id}","starred_url":"https://api.github.com/users/kentonl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kentonl/subscriptions","organizations_url":"https://api.github.com/users/kentonl/orgs","repos_url":"https://api.github.com/users/kentonl/repos","events_url":"https://api.github.com/users/kentonl/events{/privacy}","received_events_url":"https://api.github.com/users/kentonl/received_events","type":"User","site_admin":false},"created_at":"2016-01-12T04:50:11Z","updated_at":"2016-01-12T04:50:11Z","body":"I switched to `with_rank` for the shape checking. Looking cleaner now, thanks!\r\n\r\nThis is necessary because of this shape check in the `reverse_sequence` op: https://github.com/tensorflow/tensorflow/blob/d1b8333effdcb031e6e34a2835a2f1c877fdd79b/tensorflow/python/ops/array_ops.py#L1014. We can't promote the input to rank 2 there, because `tf.reverse_sequence` does not make assumptions about the exact rank of the input."}}