Skip to content

Conversation

rhshadrach
Copy link
Member

Part of #46560

@rhshadrach rhshadrach added Enhancement Window rolling, ewma, expanding Nuisance Columns Identifying/Dropping nuisance columns in reductions, groupby.add, DataFrame.apply labels Jun 7, 2022
@rhshadrach rhshadrach added this to the 1.5 milestone Jun 7, 2022
@rhshadrach rhshadrach mentioned this pull request Jun 7, 2022
9 tasks
@rhshadrach rhshadrach force-pushed the numeric_only_window branch from e99e35a to 8ac452c Compare June 7, 2022 03:00
Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might have missed it, but could you add tests for cov/corr as well (in test_pariwise.py). Those are not covered in arithmetic_win_operators.

@rhshadrach
Copy link
Member Author

Doh - thanks @mroeschke; test added and implementation fixed for Series case.

@mroeschke mroeschke merged commit 730b307 into pandas-dev:main Jun 8, 2022
@mroeschke
Copy link
Member

Awesome thanks @rhshadrach. Was this the final change for #46560?

@rhshadrach
Copy link
Member Author

rhshadrach commented Jun 8, 2022

@mroeschke - I believe so, but going to be reviewing the API manually to double check things before closing that issue. Will report there and manually close if done.

@rhshadrach rhshadrach deleted the numeric_only_window branch June 8, 2022 21:07
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
* ENH: Add numeric_only to window ops

* Fix corr/cov for Series; add tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Nuisance Columns Identifying/Dropping nuisance columns in reductions, groupby.add, DataFrame.apply Window rolling, ewma, expanding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants