Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Use return instead of break to correctly exit loop on path exist… #1312

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Merge branch 'intelligent-machine-learning:master' into master

eb3797e
Select commit
Loading
Failed to load commit list.
Open

Fix: Use return instead of break to correctly exit loop on path exist… #1312

Merge branch 'intelligent-machine-learning:master' into master
eb3797e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 5, 2024 in 1s

33.33% of diff hit (target 80.00%)

View this Pull Request on Codecov

33.33% of diff hit (target 80.00%)

Annotations

Check warning on line 628 in dlrover/python/elastic_agent/torch/ckpt_saver.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dlrover/python/elastic_agent/torch/ckpt_saver.py#L628

Added line #L628 was not covered by tests

Check warning on line 662 in dlrover/python/elastic_agent/torch/ckpt_saver.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dlrover/python/elastic_agent/torch/ckpt_saver.py#L662

Added line #L662 was not covered by tests

Check warning on line 926 in dlrover/python/elastic_agent/torch/ckpt_saver.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dlrover/python/elastic_agent/torch/ckpt_saver.py#L926

Added line #L926 was not covered by tests

Check warning on line 1086 in dlrover/python/elastic_agent/torch/ckpt_saver.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dlrover/python/elastic_agent/torch/ckpt_saver.py#L1086

Added line #L1086 was not covered by tests