-
Notifications
You must be signed in to change notification settings - Fork 57
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
which tensorflow version are you using? Type error when implement with tf v1.0 #1
Comments
tf.nn.seq2seq.embedding_attention_seq2seq
?
Same problem I have met. |
Problem is with broken backward compatibilities between two different versions of TensorFlow. Expect fix soon, the code will be updated so it can work with the newest version of tf. @EmbraceLife @jsdtwry |
Hi @Nemzy, are you waiting for a new version of tf before you can update your code? If so, that can be a while right? I wonder which previous version did you use to make your notebook work, given v1.0 is not working for the moment. I want to try it with the older version, if it can work. Or do you have to tweak even that tf version in order to make the notebook to work? If so, could you tell me how? Thanks |
@EmbraceLife Just use these functions tf.contrib.legacy_seq2seq.embedding_attention_seq2seq and tf.contrib.legacy_seq2seq.sequence_loss and change sampled_loss(logits, labels) to sampled_loss(labels, logits) (this was changed in v1.0). I think it will work with these changes. I've used v0.12. |
Thanks @Nemzy
Warning:
Error:
|
I am going to update my notebook today with tf v1.0. @EmbraceLife @jsdtwry |
looking forward to it |
Notebook has been updated (TensorFlow version r1.0). If you find any errors feel free to open new issue. I'm sorry to have kept you waiting, best regards. @EmbraceLife @jsdtwry |
thanks, it is training without having a error, given |
Hi, Thanks for this wonderful repo!
when I execute the line
it appear on my tensorflow v1.0 there is no such module, nor did v1.0 API mention it online.
Which tensorflow version are you using? How can I use the same version?
I tried to implement your code with tensorlfow v1.0 with some changes on the function names. I used those functions below:
tf.contrib.legacy_seq2seq.embedding_attention_seq2seq
andtf.contrib.legacy_seq2seq.sequence_loss
to replacetf.nn.seq2seq.embedding_attention_seq2seq
andtf.nn.seq2seq.sequence_loss
However, when I execute
loss = tf.contrib.legacy_seq2seq.sequence_loss(outputs, targets, target_weights, softmax_loss_function = sampled_loss)
, I got the following error:Could you help me to fix this error?
Thanks
The text was updated successfully, but these errors were encountered: