-
Notifications
You must be signed in to change notification settings - Fork 418
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
learn.ops.split_squeeze(1, time_steps, X) and tf.unpack() #7
Comments
Hellom @QuantScientist , This function seems to exist, however, some tweaks are needed. Other than this, did you experience other issues in updating the code for TF 1.0? I plan to convert this piece of excellent code myself, starting with this in one or two weeks... BR |
tf.nn.rnn_cell was replaced by tf.contrib.rnn. Now I am still trying to find a way to update the split_squeeze |
@tucakov , after a few simple tweaks I got this code working (updated for TF 1.0): https://github.com/akash13singh/LSTM_TimeSeries There are however quite a few warnings of functions that will become obsolete and this script is only running on 1 thread (and not on the gpu) but hopefully I will improve both of these items... |
@pusj cool! I'll have a look!!! |
@tucakov , I have now done a few further updates and added these the the repository I just created...: https://github.com/pusj/LSTM-Time-Series-Analysis-using-Tensorflow Please feel free to add comments to the two issues I've added in the issue list |
Hi,
I am upgrading the example to the latest TF.
In your comments you mention that split.squeeze should be replaced by tf.unpak, however there is no tf.unpack in the latest TF. Can you please clarify what you had in mind?
Thanks,
The text was updated successfully, but these errors were encountered: