Skip to content

Conversation

@GuoxiaWang
Copy link
Contributor

@GuoxiaWang GuoxiaWang commented Feb 27, 2022

PR types

New features

PR changes

OPs

Describe

add MasterParam and MasterParamOut for sparse_momentum op in pybind/op_function_generator.h

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@CLAassistant
Copy link

CLAassistant commented Feb 27, 2022

CLA assistant check
All committers have signed the CLA.

@GuoxiaWang GuoxiaWang closed this Feb 28, 2022
@GuoxiaWang GuoxiaWang reopened this Feb 28, 2022
@GuoxiaWang GuoxiaWang closed this Feb 28, 2022
@GuoxiaWang GuoxiaWang reopened this Feb 28, 2022
@GuoxiaWang GuoxiaWang force-pushed the master_param_sparse_momentum branch from 5b00ba9 to 05e44c7 Compare February 28, 2022 11:01
Copy link

@sandyhouse sandyhouse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@zhiqiu zhiqiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sandyhouse sandyhouse merged commit 9de7989 into PaddlePaddle:develop Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants