Skip to content

Conversation

@liym27
Copy link
Contributor

@liym27 liym27 commented Jul 8, 2025

PR Category

Auto Parallel

PR Types

Bug fixes

Description

fix bug of placements in all2all for moe.

  • Use sub-placements instead of placements to calculate local shape
    PCard-86802
@liym27 liym27 requested a review from pkuzyc July 8, 2025 06:25
@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.

@liym27 liym27 force-pushed the fix_all2all_subplacements_0625 branch from c196603 to 34eecf2 Compare July 9, 2025 12:07
Copy link
Contributor

@pkuzyc pkuzyc left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Additional details and impacted files
@@ Coverage Diff @@ ## develop #73894 +/- ## =========================================== Coverage ? 100.00% =========================================== Files ? 1 Lines ? 1 Branches ? 0 =========================================== Hits ? 1 Misses ? 0 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.
@liym27 liym27 merged commit 2ce32c9 into PaddlePaddle:develop Jul 10, 2025
74 of 77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants