search

pender / chatbot-rnn

A toy chatbot powered by deep learning and trained on data from Reddit
MIT License
899 stars 370 forks source link

state_is_tuple=True: Fix training on 0.12 #4

Closed julien-c closed 7 years ago

julien-c commented 7 years ago

This parallels the changes made to char-rnn-tensorflow here: https://github.com/sherjilozair/char-rnn-tensorflow/commit/991704e68632b57c1591651ce8183e5a374ca44a which in turn parallels changes made to TF tutorial ptb_word_lm.py

pender commented 7 years ago

Isn't this already the default value of the argument? Is there a style reason why it is preferred to make the argument explicit?

julien-c commented 7 years ago

Not sure, but it worked in my install (was failing without the change)

pender commented 7 years ago

Odd! Okay, well it can't hurt -- will merge it in. Thanks for the PR!