Skip to content

Conversation

@zhangyuqin1998
Copy link
Contributor

@zhangyuqin1998 zhangyuqin1998 commented Jul 8, 2025

PR Category

Distributed Strategy

PR Types

New features

Description

Support param and moment sync for pp shared layer

Pcard-76459

@paddle-bot
Copy link

paddle-bot bot commented Jul 8, 2025

你的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.

@zhangyuqin1998
Copy link
Contributor Author

/re-run all-failed

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

@zhangyuqin1998
Copy link
Contributor Author

/re-run all-failed

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 75.28090% with 22 lines in your changes missing coverage. Please review.

Please upload report for BASE (develop@d7127bd). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...ers/dygraph_optimizer/hybrid_parallel_optimizer.py 74.07% 21 Missing ⚠️
python/paddle/base/framework.py 75.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (75.28%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@ Coverage Diff @@ ## develop #73913 +/- ## ========================================== Coverage ? 75.28% ========================================== Files ? 3 Lines ? 89 Branches ? 0 ========================================== Hits ? 67 Misses ? 22 Partials ? 0 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
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 merged commit 73c1316 into PaddlePaddle:develop Jul 11, 2025
69 of 72 checks passed
@zhangyuqin1998 zhangyuqin1998 deleted the pp_sync branch October 25, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants