Skip to content

FIX: series.fillna doesn't shallow copy if len(series) == 0 #32733

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

Conversation

SaturnFromTitan
Copy link
Contributor

@SaturnFromTitan SaturnFromTitan commented Mar 15, 2020

@SaturnFromTitan
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 32733 in repo pandas-dev/pandas

@SaturnFromTitan SaturnFromTitan force-pushed the 32543-obj-fillna-fill_value-doesn-t-shallow-copy-if-obj-is-an-empty-series branch from 0d6f740 to 93797f4 Compare March 15, 2020 22:05
Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

looks ok, ping on green.

@jreback jreback added the Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate label Mar 16, 2020
@jreback jreback added this to the 1.1 milestone Mar 16, 2020
@SaturnFromTitan
Copy link
Contributor Author

@jreback Adapted the isnew section and CI is green

@SaturnFromTitan
Copy link
Contributor Author

SaturnFromTitan commented Mar 16, 2020

@jreback btw, is there any chance I could get privileges to run /azp run?

@jreback
Copy link
Contributor

jreback commented Mar 16, 2020

@SaturnFromTitan added you to pandas-triage

you can now label & close issues (i think u can prob trigger CI not sure about that)

@SaturnFromTitan
Copy link
Contributor Author

sweet, thanks a lot @jreback!

@WillAyd WillAyd merged commit 2b17609 into pandas-dev:master Mar 16, 2020
@WillAyd
Copy link
Member

WillAyd commented Mar 16, 2020

Great thanks @SaturnFromTitan and congrats on the triage role - well deserved!

@SaturnFromTitan
Copy link
Contributor Author

Thanks for the kind words @WillAyd :)

SeeminSyed pushed a commit to CSCD01-team01/pandas that referenced this pull request Mar 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

obj.fillna(fill_value) doesn't shallow copy if obj is an empty Series
3 participants