lamtram icon indicating copy to clipboard operation
lamtram copied to clipboard
verified publisher icon neubig

Trainer::update_epoch has been deprecated

Open shirayu opened this issue 8 years ago • 0 comments

Following to the https://github.com/clab/dynet/pull/695 , trainer->update_epoch() in src/lamtram/lamtram-train.cc should be removed.

Is it ok to simply insert the following line?

trainer->learning_rate /= (1 - rate_decay_)

What is the difference between trainer->learning_rate and learning_rate declared at https://github.com/neubig/lamtram/blob/master/src/lamtram/lamtram-train.cc#L929 ?

shirayu avatar Nov 01 '17 05:11 shirayu