onomatet commented on issue #17207: LR schedulers do not work in R
URL: 
https://github.com/apache/incubator-mxnet/issues/17207#issuecomment-576714416
 
 
   @hetong007 Thank you for your reply!
   
   Honestly, I don't see how would it help. lr schedulers are called properly 
on every update here
   
https://github.com/apache/incubator-mxnet/blob/25505e9da4ea24ce37f1e60916d1afc3fcd15300/R-package/R/optimizer.R#L86
   and they modify the `lr` parameter in the `sgd` namespace.
   
   From my understanding, however, it doesn't matter since `lr` is a property 
of the `mx.symbol.sgd_mom_update` which cannot be changed after the 
correspondig executor `exec` is created in
   
https://github.com/apache/incubator-mxnet/blob/25505e9da4ea24ce37f1e60916d1afc3fcd15300/R-package/R/optimizer.R#L79
   (or can it?)

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to