-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@ThomasDelteil - Thanks for fixing this |
This comment is ignored: #12171 (comment) |
@mxnet-label-bot update [Doc, pr-awaiting-merge] |
@ThomasDelteil The issue mentioned #12171 talks about doc fix needed and the bug in optimizer. This PR only takes care of doc fix. |
@ThomasDelteil Could you please take a look into comments by @Roshrini |
I've updated the description. The issue is left open. |
* change docs lbsgd * fix more
* change docs lbsgd * fix more
* change docs lbsgd * fix more
* change docs lbsgd * fix more
* change docs lbsgd * fix more
Description
fix doc in #12171