Skip to content

BUG: Fix skipna default value in method signatures #43980

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 1 commit into from
Oct 16, 2021
Merged

BUG: Fix skipna default value in method signatures #43980

merged 1 commit into from
Oct 16, 2021

Conversation

jhlegarreta
Copy link
Contributor

Set skipna default values to True so that the value matches the
purpose of the variable and its docstring.

@jhlegarreta
Copy link
Contributor Author

I did not change the mad method, since its signature matches the docstring and the implementation. Changing that one would require a deprecation cycle I guess; if not, let me know and I will amend the commit.

@jreback
Copy link
Contributor

jreback commented Oct 12, 2021

I did not change the mad method, since its signature matches the docstring and the implementation. Changing that one would require a deprecation cycle I guess; if not, let me know and I will amend the commit.

yeah this is fine

@jreback jreback added this to the 1.4 milestone Oct 12, 2021
@jreback
Copy link
Contributor

jreback commented Oct 12, 2021

looks fine. @jbrockmendel ok here?

@jbrockmendel
Copy link
Member

could remove the if skipna is None in _stat_function

@jhlegarreta
Copy link
Contributor Author

@jreback @jbrockmendel any suggestion as to what I should do concerning #43980 (comment) ? Thanks.

@jreback
Copy link
Contributor

jreback commented Oct 16, 2021

@jreback @jbrockmendel any suggestion as to what I should do concerning #43980 (comment) ? Thanks.

see if you break anything by changing them. IIRC we did this for some backwards compat but that was quite a while ago.

Set `skipna` default values to `True` so that the value matches the
purpose of the variable and its docstring.
@jhlegarreta
Copy link
Contributor Author

see if you break anything by changing them. IIRC we did this for some backwards compat but that was quite a while ago.

👍 Push-forced in e15a1ce. Let's see what the CIs say.

@jreback jreback merged commit 1d530e4 into pandas-dev:master Oct 16, 2021
@jreback
Copy link
Contributor

jreback commented Oct 16, 2021

thanks @jhlegarreta very nice!

@jhlegarreta jhlegarreta deleted the FixSkipnaDefaultValueInMethodSignatures branch October 16, 2021 17:30
@phofl phofl mentioned this pull request Nov 23, 2021
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: Dataframe.skew invalid function signature for "skipna"
4 participants