-
Notifications
You must be signed in to change notification settings - Fork 13
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
Suppress or fix warnings #1272
Suppress or fix warnings #1272
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main-dev #1272 +/- ##
============================================
- Coverage 79.04% 78.95% -0.09%
============================================
Files 136 136
Lines 20786 20835 +49
============================================
+ Hits 16431 16451 +20
- Misses 4355 4384 +29
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
b661ab7
to
22b8697
Compare
22b8697
to
788ea9e
Compare
@@ -52,6 +53,10 @@ def test_unitconv_sfc_conc(dummy_data): | |||
assert np.all(result == pytest.approx(1.99796663, 1e-4)) | |||
|
|||
|
|||
@ignore_warnings( |
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.
Maybe add a TODO: Change to update pandas without breaking other tests
Change Summary
This PR reduces the number of warnings in the log by either suppressing them, fixing the underlying issue or ignoring them depending on the warning:
stats/implementations.py
, "invalid value encountered in divide" (caused by division by zero) andConstantInputWarning
(correlation undefined for constant arrays) are ignored.Related issue number
#1066
Checklist