search

tensorflow / skflow

Simplified interface for TensorFlow (mimicking Scikit Learn) for Deep Learning
Apache License 2.0
3.18k stars 439 forks source link

fix of the rnn_seq2seq in order to be compatible with tensorflow 0.7 #123

Closed ziky90 closed 8 years ago

ziky90 commented 8 years ago

This PR is addition to #119 and fixes issue #121

Since tensorflow 0.7. tf.nn.rnn() does return only the last state, so seq2seq_ops.py file needs this update where returned element is treated as tensor, not list of tensors.

codecov-io commented 8 years ago

Current coverage is 90.38%

Merging #123 into master will not affect coverage as of bf50d54

@@            master    #123   diff @@
======================================
  Files           27      27       
  Stmts         1061    1061       
  Branches       164     164       
  Methods          0       0       
======================================
  Hit            959     959       
  Partial         50      50       
  Missed          52      52

Review entire Coverage Diff as of bf50d54

Powered by Codecov. Updated on successful CI builds.

terrytangyuan commented 8 years ago

Thanks!