access2rohit commented on a change in pull request #16715: Lamb optimizer update URL: https://github.com/apache/incubator-mxnet/pull/16715#discussion_r346107803
########## File path: src/operator/optimizer_op.cc ########## @@ -921,5 +923,39 @@ Note that non-zero values for the weight decay option are not supported. .add_argument("history", "NDArray-or-Symbol", "History") .add_arguments(AdagradParam::__FIELDS__()); +NNVM_REGISTER_OP(lamb_update) +.describe(R"code(Update function for lamb optimizer. +)code" ADD_FILELINE) +.set_num_inputs(4) +.set_num_outputs(1) +.set_attr_parser(ParamParser<LAMBParam>) +.set_attr<mxnet::FInferShape>("FInferShape", ElemwiseShape<4, 1>) +.set_attr<nnvm::FInferType>("FInferType", ElemwiseType<4, 1>) +.set_attr<FCompute>("FCompute<cpu>", LAMBUpdate<cpu>) +.set_attr<nnvm::FMutateInputs>("FMutateInputs", + [](const nnvm::NodeAttrs& attrs) { + return std::vector<uint32_t>{2, 3}; + }) +.add_argument("weight", "NDArray-or-Symbol", "Weight") +.add_argument("grad", "NDArray-or-Symbol", "Gradient") +.add_argument("mean", "NDArray-or-Symbol", "Moving mean") +.add_argument("var", "NDArray-or-Symbol", "Moving variance") +.add_arguments(LAMBParam::__FIELDS__()); + +NNVM_REGISTER_OP(lamb_weight_update) +.describe(R"code(Update function for lamb optimizer. +)code" ADD_FILELINE) +.set_num_inputs(4) +.set_num_outputs(1) +.set_attr_parser(ParamParser<LAMBWeightParam>) +.set_attr<mxnet::FInferShape>("FInferShape", LambWeightShape) +.set_attr<nnvm::FInferType>("FInferType", ElemwiseType<4, 1>) +.set_attr<FCompute>("FCompute<cpu>", LAMBWeightUpdate<cpu>) +.add_argument("weight", "NDArray-or-Symbol", "Weight") +.add_argument("g", "NDArray-or-Symbol", "g") Review comment: Can you suggest a link or what should I write here. I have followed this: https://towardsdatascience.com/an-intuitive-understanding-of-the-lamb-optimizer-46f8c0ae4866 Not sure how should I describe `g` ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services