Skip to content

DEPR: Enforce deprecation of groupby(..., axis=1) #57186

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 5 commits into from
Feb 1, 2024

Conversation

rhshadrach
Copy link
Member

@rhshadrach rhshadrach commented Jan 31, 2024

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added type annotations to new arguments/methods/functions.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

Enforcing #51203

@rhshadrach rhshadrach added Groupby Deprecate Functionality to remove in pandas labels Jan 31, 2024
@rhshadrach rhshadrach added this to the 3.0 milestone Jan 31, 2024
@@ -102,8 +102,10 @@ Deprecations
Removal of prior version deprecations/changes
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- Removed :meth:`DataFrameGroupby.fillna` and :meth:`SeriesGroupBy.fillna` (:issue:`55719`)
- Removed ``axis`` argument from :meth:`DataFrame.groupby` and :meth:`Series.groupby` (:issue:`51203`)
Copy link
Member

Choose a reason for hiding this comment

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

Could you mention rolling and resample too?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, done.

Comment on lines +1524 to 1526
# We have multi-block tests
# e.g. test_rank_min_int, test_cython_transform_frame
# test_transform_numeric_ret
Copy link
Member Author

Choose a reason for hiding this comment

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

I think this comment can be fully removed now, but wanted to make sure.

Copy link
Member

Choose a reason for hiding this comment

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

I think so too. Can be done in a follow up

# Conflicts:
#	doc/source/whatsnew/v3.0.0.rst
#	pandas/core/groupby/generic.py
@rhshadrach rhshadrach marked this pull request as ready for review February 1, 2024 04:13
@mroeschke mroeschke merged commit 24f7db7 into pandas-dev:main Feb 1, 2024
@mroeschke
Copy link
Member

Thanks @rhshadrach

@rhshadrach rhshadrach deleted the enf_groupby_axis_1 branch February 1, 2024 21:37
pmhatre1 pushed a commit to pmhatre1/pandas-pmhatre1 that referenced this pull request May 7, 2024
* DEPR: Enforce deprecation of groupby(..., axis=1)

* More removals and cleanups

* whatsnew
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecate Functionality to remove in pandas Groupby
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants