bnlstm icon indicating copy to clipboard operation
bnlstm copied to clipboard

Update for tensorflow 1.12.0

Open pren1 opened this issue 6 years ago • 2 comments

Just some version problems, and I believe you can also find solutions easily by yourself :D

changes:

scalar_summary should be renamed to tf.summary.scalar
tf.histogram_summary should be renamed to tf.summary.histogram
tf.merge_summary should be renamed to tf.summary.merge
tf.train.SummaryWriter should be renamed to tf.summary.FileWriter

pren1 avatar Dec 21 '18 00:12 pren1

Thank you for your implementation!

pren1 avatar Dec 21 '18 00:12 pren1

By the way, to use the vanilla LSTM here on tensorflow 1.12.0, change:

concat = tf.concat(1, [x, h]) W_both = tf.concat(0, [W_xh, W_hh])

to

concat = tf.concat([x, h], 1) W_both = tf.concat([W_xh, W_hh], 0)

pren1 avatar Jan 01 '19 19:01 pren1