intelligent-machine-learning / dlrover

DLRover: An Automatic Distributed Deep Learning System
Other
1.22k stars 153 forks source link

Skip 'should early stop' for non all reduce job. #1238

Closed BalaBalaYi closed 1 month ago

BalaBalaYi commented 1 month ago

What changes were proposed in this pull request?

To skip 'should early stop' for non all reduce job.

Why are the changes needed?

Optimization for different case.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

UT.

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.49%. Comparing base (e50fe48) to head (ae74bde). Report is 7 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1238 +/- ## ========================================== + Coverage 80.47% 80.49% +0.01% ========================================== Files 217 217 Lines 19599 19612 +13 ========================================== + Hits 15773 15786 +13 Misses 3826 3826 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.