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.

Sign up for GitHub

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

Jump to bottom

`model_saving_strategy=best` does not work #514

Merged

WenjieDu merged 1 commit into dev from (fix)model_saving

Sep 22, 2024

Owner

WenjieDu commented Sep 19, 2024

What does this PR do?

fixing Model saving bug #464;

Before submitting

Was this discussed/approved via a GitHub issue? Please add a link to it if that's the case.


          fix: model_saving_strategy=best does not work;

f28353a

Collaborator

coveralls commented Sep 19, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 10932122525

Details

39 of 39 (100.0%) changed or added relevant lines in 39 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 83.346%

Files with Coverage Reduction	New Missed Lines	%
pypots/nn/modules/fedformer/layers.py	2	45.99%
pypots/cli/doc.py	3	82.28%

Totals
Change from base Build 10833804335:	-0.02%
Covered Lines:	11195
Relevant Lines:	13432

💛 - Coveralls

WenjieDu merged commit d8ade79 into dev

9 checks passed

WenjieDu deleted the (fix)model_saving branch

September 25, 2024 06:58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet