Skip to content

Conversation

@waliwali777
Copy link
Contributor

@waliwali777 waliwali777 commented Jul 2, 2025

Before submitting

  • Lint code. If there are lint issues, please format the code first.
# Install and register `pre-commit` in the project folder pip install pre-commit && pre-commit install # Process previous code files separately pre-commit run --file XXXX.py
  • Add test cases into tests folder. If there are codecov issues, please add tests cases first.

PR types

Others

PR changes

Others

Description

之前因为 paddle 中默认使用 FLAGS_fuse_reducescatter_in_opt 优化( PaddlePaddle/Paddle#73744 ),需要在 CI 中暂时关闭 sharding 相关单测( #10791 )
该 PR 修改sharding 单测的 loss_base,并再次打开
并移除 benchamrk 中所有 FLAGS_fuse_reducescatter_in_opt

@paddle-bot
Copy link

paddle-bot bot commented Jul 2, 2025

Thanks for your contribution!

@codecov
Copy link

codecov bot commented Jul 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.76%. Comparing base (78741f9) to head (afb5ee5).
Report is 2 commits behind head on develop.

Current head afb5ee5 differs from pull request most recent head 796c320

Please upload reports for the commit 796c320 to get more accurate results.

❌ Your project status has failed because the head coverage (46.76%) is below the target coverage (58.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@ Coverage Diff @@ ## develop #10793 +/- ## =========================================== - Coverage 46.80% 46.76% -0.05%  =========================================== Files 802 802 Lines 133728 133882 +154 =========================================== + Hits 62597 62608 +11  - Misses 71131 71274 +143 

☔ 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.
@waliwali777 waliwali777 changed the title Open dynamic sd test [AutoParallel] Open dynamic sharding CI test Jul 2, 2025
@waliwali777 waliwali777 force-pushed the open_dynamic_sd_test branch 2 times, most recently from 0c65540 to a55742c Compare July 9, 2025 05:07
@waliwali777 waliwali777 force-pushed the open_dynamic_sd_test branch from 55f9b00 to afb5ee5 Compare July 9, 2025 07:59
@waliwali777
Copy link
Contributor Author

/re-run all-failed

Copy link
Contributor

@liym27 liym27 left a comment

Choose a reason for hiding this comment

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

LGTM

@wawltor wawltor merged commit 15d49bb into PaddlePaddle:develop Jul 10, 2025
7 of 10 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