codecov[bot]
commented
1 month ago Codecov Report
Attention: Patch coverage is 91.66667% with 2 lines in your changes missing coverage. Please review.
Project coverage is 80.60%. Comparing base (
211903e) to head (f814901). Report is 4 commits behind head on master.
| Files | Patch % | Lines |
|---|---|---|
| ...lrover/python/tests/test_elastic_training_agent.py | 89.47% | 2 Missing :warning: |
Additional details and impacted files
```diff @@ Coverage Diff @@ ## master #1253 +/- ## ========================================== + Coverage 80.41% 80.60% +0.18% ========================================== Files 218 218 Lines 19800 19855 +55 ========================================== + Hits 15923 16004 +81 + Misses 3877 3851 -26 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What changes were proposed in this pull request?
Add judgement on 'exclude_stragger' config.
Why are the changes needed?
Should consider the 'exclude_stragger' config if there is stragglers only.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
UT.