-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
selected numeric data before correlation #18651
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
Conversation
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.
lgtm. whatsnew for 0.21.1 is good. ping on green.
pandas/tests/frame/test_analytics.py
Outdated
@@ -240,6 +240,15 @@ def test_corrwith_matches_corrcoef(self): | |||
tm.assert_almost_equal(c1, c2) | |||
assert c1 < 1 | |||
|
|||
def test_corrwith_mixed_dtypes(self): | |||
df = pd.DataFrame({'a': [1, 4, 3, 2], 'b': [4, 6, 7, 3], |
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.
add the issue number as a comment
actually do this for 0.22 |
Codecov Report
@@ Coverage Diff @@
## master #18651 +/- ##
==========================================
- Coverage 91.6% 91.57% -0.03%
==========================================
Files 153 153
Lines 51250 51221 -29
==========================================
- Hits 46947 46908 -39
- Misses 4303 4313 +10
Continue to review full report at Codecov.
|
thanks! |
git diff upstream/master -u -- "*.py" | flake8 --diff
I select the numeric data before the calculation happens with a Series.
No whatsnew entry yet.