{"action":"created","issue":{"url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials/issues/46","repository_url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials","labels_url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials/issues/46/labels{/name}","comments_url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials/issues/46/comments","events_url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials/issues/46/events","html_url":"https://github.com/nlintz/TensorFlow-Tutorials/pull/46","id":166944142,"number":46,"title":"Fixed tensorflow 0.9 API compatibility","user":{"login":"j-min","id":18069263,"avatar_url":"https://avatars.githubusercontent.com/u/18069263?v=3","gravatar_id":"","url":"https://api.github.com/users/j-min","html_url":"https://github.com/j-min","followers_url":"https://api.github.com/users/j-min/followers","following_url":"https://api.github.com/users/j-min/following{/other_user}","gists_url":"https://api.github.com/users/j-min/gists{/gist_id}","starred_url":"https://api.github.com/users/j-min/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/j-min/subscriptions","organizations_url":"https://api.github.com/users/j-min/orgs","repos_url":"https://api.github.com/users/j-min/repos","events_url":"https://api.github.com/users/j-min/events{/privacy}","received_events_url":"https://api.github.com/users/j-min/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-07-22T00:00:54Z","updated_at":"2016-07-22T07:13:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials/pulls/46","html_url":"https://github.com/nlintz/TensorFlow-Tutorials/pull/46","diff_url":"https://github.com/nlintz/TensorFlow-Tutorials/pull/46.diff","patch_url":"https://github.com/nlintz/TensorFlow-Tutorials/pull/46.patch"},"body":"Fixed tensorflow 0.9 API compatibility\r\n1. replaced \"from tensorflow.models.rnn import rnn, rnn_cell\" with \"tf.nn.rnn_cell\"\r\n2. added state_is_tuple=True (https://www.tensorflow.org/versions/r0.9/api_docs/python/nn.html#recurrent-neural-networks)\r\n3. using dtype=tf.float32 instead of init_state"},"comment":{"url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials/issues/comments/234471414","html_url":"https://github.com/nlintz/TensorFlow-Tutorials/pull/46#issuecomment-234471414","issue_url":"https://api.github.com/repos/nlintz/TensorFlow-Tutorials/issues/46","id":234471414,"user":{"login":"j-min","id":18069263,"avatar_url":"https://avatars.githubusercontent.com/u/18069263?v=3","gravatar_id":"","url":"https://api.github.com/users/j-min","html_url":"https://github.com/j-min","followers_url":"https://api.github.com/users/j-min/followers","following_url":"https://api.github.com/users/j-min/following{/other_user}","gists_url":"https://api.github.com/users/j-min/gists{/gist_id}","starred_url":"https://api.github.com/users/j-min/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/j-min/subscriptions","organizations_url":"https://api.github.com/users/j-min/orgs","repos_url":"https://api.github.com/users/j-min/repos","events_url":"https://api.github.com/users/j-min/events{/privacy}","received_events_url":"https://api.github.com/users/j-min/received_events","type":"User","site_admin":false},"created_at":"2016-07-22T07:13:16Z","updated_at":"2016-07-22T07:13:16Z","body":"According to https://github.com/nlintz/TensorFlow-Tutorials/blob/master/.travis.yml\r\nthis repository is based on tensorflow 0.8.0 environment where rnn modules do not accept 'state_is_tuple' argument.\r\n\r\n'  - if [[ \"$TRAVIS_PYTHON_VERSION\" == \"2.7\" ]]; then\r\n      pip install https://storage.googleapis.com/tensorflow/linux/cpu/tensorflow-0.8.0-cp27-none-linux_x86_64.whl;\r\n    elif [[ \"$TRAVIS_PYTHON_VERSION\" == \"3.4\" ]]; then\r\n      pip install https://storage.googleapis.com/tensorflow/linux/cpu/tensorflow-0.8.0-cp34-cp34m-linux_x86_64.whl;'\r\n\r\nOne of the reason I made this commit is that not using 'state_is_tuple' makes error\r\n`WARNING:tensorflow:<tensorflow.python.ops.rnn_cell.BasicLSTMCell object at 0x101e360b8>:\r\nUsing a concatenated state is slower and will soon be deprecated.  Use state_is_tuple=True.`\r\n\r\nCan we update our travis environment?"}}
{"actor":{"display_login":"j-min"}}