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

Fix unintended overwrite saving #516

Merged

WenjieDu merged 1 commit into dev from (fix)saving_overwrite

Sep 19, 2024

Owner

WenjieDu commented Sep 19, 2024

What does this PR do?

fixing Unintended saving success when model file already exists #515;

Before submitting

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


          fix: unintended overwrite saving;

ced431f

Collaborator

coveralls commented Sep 19, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 10932379202

Details

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 83.34%

Files with Coverage Reduction	New Missed Lines	%
pypots/imputation/csdi/model.py	1	84.85%
pypots/cli/doc.py	3	82.28%

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

💛 - Coveralls

WenjieDu merged commit 5263088 into dev

9 checks passed

WenjieDu deleted the (fix)saving_overwrite branch

September 19, 2024 03:00

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

Labels

None yet