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.

Sign up for GitHub

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

Jump to bottom

【Distributed】Delete scale_and_split_grads function #64154

Merged
merged 2 commits into from May 11, 2024
Merged

【Distributed】Delete scale_and_split_grads function #64154

merged 2 commits into from May 11, 2024

Conversation

heavyrain-lzy
Copy link
Contributor

@heavyrain-lzy heavyrain-lzy commented May 9, 2024 •
edited

PR Category

Others

PR Types

Bug fixes

Description

Pcard-76459
Repalce the function scale_and_split_grads using scale_grads.

@paddle-bot Paddle-bot
Copy link

paddle-bot bot commented May 9, 2024

你的PR提交成功，感谢你对开源项目的贡献!
请关注后续CI自动化测试结果，详情请参考Paddle-CI手册。
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

ForFishes
ForFishes previously approved these changes May 10, 2024
View reviewed changes
Copy link
Member

@ForFishes ForFishes left a comment

Choose a reason for hiding this comment

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

LGTM

ForFishes
ForFishes approved these changes May 11, 2024
View reviewed changes
Copy link
Member

@ForFishes ForFishes left a comment

Choose a reason for hiding this comment

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

LGTM

@heavyrain-lzy heavyrain-lzy merged commit 6b63e5e into PaddlePaddle:develop May 11, 2024
30 of 31 checks passed
co63oc pushed a commit to co63oc/Paddle that referenced this pull request May 11, 2024
@heavyrain-lzy @co63oc
【Distributed】Delete scale_and_split_grads function (PaddlePaddle#64154
4d3d62b 
)

* delete scale_and_split_grads function

* fix image_classification_resnet.inference.model.tgz
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ForFishes ForFishes ForFishes approved these changes

@SereneZephyrus SereneZephyrus SereneZephyrus approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@heavyrain-lzy @ForFishes @SereneZephyrus