Skip to content

DOC: Add docs for DataFrameGroupBy.fillna and SeriesGroupBy.fillna #48771

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 2 commits into from
Sep 26, 2022

Conversation

rhshadrach
Copy link
Member

  • 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.

@rhshadrach rhshadrach added Enhancement Docs Groupby Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate labels Sep 25, 2022
@rhshadrach rhshadrach added this to the 1.6 milestone Sep 25, 2022
@@ -6730,9 +6730,9 @@ def fillna(
each index (for a Series) or column (for a DataFrame). Values not
in the dict/Series/DataFrame will not be filled. This value cannot
be a list.
method : {{'backfill', 'bfill', 'pad', 'ffill', None}}, default None
Copy link
Member Author

Choose a reason for hiding this comment

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

pad is deprecated, so I removed from the fillna docstrings. backfill is deprecated in groupby but not DataFrame or Series. I'll look into this for a followup.

Comment on lines +781 to +782
inplace : bool, default False
Broken. Do not set to True.
Copy link
Member Author

Choose a reason for hiding this comment

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

For both DataFrameGroupBy and SeriesGroupBy, using inplace=True doesn't modify the source and returns an empty DataFrame or a Series filled with None respectively. I plan to deprecate this in a followup.

@mroeschke mroeschke merged commit 6c7507f into pandas-dev:main Sep 26, 2022
@mroeschke
Copy link
Member

Thanks @rhshadrach

@rhshadrach rhshadrach deleted the groupby_fillna_docs branch September 26, 2022 20:36
@mroeschke mroeschke modified the milestones: 1.6, 2.0 Oct 13, 2022
noatamir pushed a commit to noatamir/pandas that referenced this pull request Nov 9, 2022
…andas-dev#48771)

* DOC: Add docs for DataFrameGroupBy.fillna and SeriesGroupBy.fillna

* fixup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Enhancement Groupby Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants