-
Notifications
You must be signed in to change notification settings - Fork 18
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
MESMER-X: Test distrib_cov
#540
MESMER-X: Test distrib_cov
#540
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #540 +/- ##
==========================================
+ Coverage 62.00% 62.74% +0.74%
==========================================
Files 50 50
Lines 3561 3554 -7
==========================================
+ Hits 2208 2230 +22
+ Misses 1353 1324 -29
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks that looks good.
Ahh sorry, this is not at all done yet, I just pushed it to get the coverage report 😅 |
Ahh well looks good already 😉 |
Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com>
Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com>
Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com>
Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com>
Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com>
for more information, see https://pre-commit.ci
@mathause, I added two tests for the weighting but only very basic ones. I am merging without another review and we can change something afterwards if we want to. |
CHANGELOG.rst