search

intelligent-machine-learning / dlrover

DLRover: An Automatic Distributed Deep Learning System
Other
1.27k stars 167 forks source link

fix a bug in infer method #1340

Open jlsong01 opened 1 day ago

jlsong01 commented 1 day ago

What changes were proposed in this pull request?

a bugfix

Why are the changes needed?

the parameter of operator.infer() SHOULD be a list with one element [inference], while NOT the whole list of inferences, because inferences may include the inference which does not match the currrent operator.

Does this PR introduce any user-facing change?

No

How was this patch tested?

pre-commit test and unittest both passed

codecov[bot] commented 1 day ago

Codecov Report

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

Project coverage is 81.11%. Comparing base (ec94ab6) to head (744c11b).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1340 +/- ## ========================================== - Coverage 81.15% 81.11% -0.04% ========================================== Files 231 231 Lines 21965 21965 ========================================== - Hits 17825 17817 -8 - Misses 4140 4148 +8 ```

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