Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support dp for class_center_sample and margin_cross_entropy #39852

Merged
merged 4 commits into from
Mar 24, 2022

Conversation

GuoxiaWang
Copy link
Contributor

@GuoxiaWang GuoxiaWang commented Feb 23, 2022

PR types

New features

PR changes

APIs

Describe

support data parallel for class_center_sample and margin_cross_entropy....

@CLAassistant
Copy link

CLAassistant commented Feb 23, 2022

CLA assistant check
All committers have signed the CLA.

@paddle-bot-old
Copy link

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

add unittest

fix unittest

fix unittest
@paddle-bot-old
Copy link

paddle-bot-old bot commented Mar 8, 2022

Sorry to inform you that a9c1e82's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@GuoxiaWang GuoxiaWang closed this Mar 23, 2022
@GuoxiaWang GuoxiaWang reopened this Mar 23, 2022
@GuoxiaWang GuoxiaWang changed the title support dp for class_center_sample, margin_cross_entropy support dp for class_center_sample and margin_cross_entropy Mar 23, 2022
@GuoxiaWang GuoxiaWang closed this Mar 23, 2022
@GuoxiaWang GuoxiaWang reopened this Mar 23, 2022
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

@sandyhouse sandyhouse merged commit bff9e28 into PaddlePaddle:develop Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants