buriburisuri / sugartensor

A slim tensorflow wrapper that provides syntactic sugar for tensor variables. This library will be helpful for practical deep learning researchers not beginners.
MIT License
372 stars 63 forks source link

Hardcoded Learning rate #40

Open StefanAndjelic opened 7 years ago

StefanAndjelic commented 7 years ago

https://github.com/buriburisuri/sugartensor/blob/d2c039954777c7fbe3eb0c2ae40c45c9854deb40/sugartensor/sg_train.py#L52

I've been scratching my head about this line. Shouldn't that be opt.lr instead, or is there a reason why it's been hardcoded to 0.01?

GabrielLin commented 6 years ago

I found the same issue. I think it should be changed to the given parameter.