Skip to content

BUG: fix rolling with centering and axis=1 (#46135) #46265

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

Merged
merged 15 commits into from
May 6, 2022

Conversation

jyuv
Copy link
Contributor

@jyuv jyuv commented Mar 8, 2022

@mroeschke mroeschke added the Window rolling, ewma, expanding label Mar 11, 2022
@jyuv
Copy link
Contributor Author

jyuv commented Mar 22, 2022

@github-actions pre-commit

@jyuv
Copy link
Contributor Author

jyuv commented Mar 23, 2022

@github-actions pre-commit

@github-actions
Copy link
Contributor

This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this.

@github-actions github-actions bot added the Stale label Apr 23, 2022
@jyuv
Copy link
Contributor Author

jyuv commented Apr 25, 2022

Hi,
maybe I'm missing something but it seemed to me that the failures in the checks are unrelated to the commited changes.
Can someone please take a look at it?

@mroeschke
Copy link
Member

You will need to merge main and push to this branch again. Looks like previously the CI didn't run all the jobs

…ibution

� Conflicts:
�	pandas/tests/window/test_win_type.py
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.

Thanks, looks pretty good. Just needs a small adjustment in the whatsnew note.

@mroeschke mroeschke removed the Stale label Apr 30, 2022
@mroeschke mroeschke added this to the 1.5 milestone May 2, 2022
@mroeschke mroeschke merged commit 44dcef4 into pandas-dev:main May 6, 2022
@mroeschke
Copy link
Member

Thanks @jyuv

@jyuv jyuv deleted the jyuv-contribution branch May 8, 2022 11:00
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
…-dev#46265)

* TST: added test for rolling with centering=True and axis=1 (pandas-dev#46135)

* BUG: fixed bug of rolling with centering=True and axis=1 (pandas-dev#46135)

* DOC: fixed bug of rolling with centering=True and axis=1 (pandas-dev#46135)

* Fixes from pre-commit [automated commit]

* TST: adapted test_rolling_center_axis_1 to support check_dtype=True

* Fixes from pre-commit [automated commit]

* TST: move test_rolling_center_axis_1 from test_rolling.py to test_win_type.py

* Fixes from pre-commit [automated commit]

* DOC: moved bug mention to rolling doc section and specify conditions (pandas-dev#46135)

Co-authored-by: Yuval <yuval,[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Window rolling, ewma, expanding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: rolling with axis=1, win_type=, and center=True raises ValueError
2 participants