Skip to content

Conversation

@TomAugspurger
Copy link
Contributor

According to
https://docs.python.org/3/library/pickle.html#object.__reduce__,

If a string is returned, the string should be interpreted as the name
of a global variable. It should be the object’s local name relative to
its module; the pickle module searches the module namespace to determine
the object’s module. This behaviour is typically useful for singletons.

Closes #31847

According to
https://docs.python.org/3/library/pickle.html#object.__reduce__,

> If a string is returned, the string should be interpreted as the name
> of a global variable. It should be the object’s local name relative to
> its module; the pickle module searches the module namespace to determine
> the object’s module. This behaviour is typically useful for singletons.

Closes pandas-dev#31847
@TomAugspurger TomAugspurger added IO Pickle read_pickle, to_pickle NA - MaskedArrays Related to pd.NA and nullable extension arrays labels Feb 19, 2020
@TomAugspurger TomAugspurger added this to the 1.0.2 milestone Feb 19, 2020
Copy link
Member

@jorisvandenbossche jorisvandenbossche left a comment

Choose a reason for hiding this comment

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

Thanks!

@jbrockmendel
Copy link
Member

@TomAugspurger can you re-push? looks like azure has stalled

@TomAugspurger
Copy link
Contributor Author

TomAugspurger commented Feb 21, 2020 via email

@jreback
Copy link
Contributor

jreback commented Feb 26, 2020

@TomAugspurger can you rebase and update (small requests)

@TomAugspurger
Copy link
Contributor Author

I think everything has been addressed.

@jorisvandenbossche jorisvandenbossche merged commit 3e5fe8e into pandas-dev:master Mar 2, 2020
@jorisvandenbossche
Copy link
Member

Thanks @TomAugspurger !

@jorisvandenbossche
Copy link
Member

@meeseeksdev backport to 1.0.x

meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Mar 2, 2020
jreback pushed a commit that referenced this pull request Mar 3, 2020
@TomAugspurger TomAugspurger deleted the 31847-pickle-NA branch March 3, 2020 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IO Pickle read_pickle, to_pickle NA - MaskedArrays Related to pd.NA and nullable extension arrays

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: pd.NA doesn't pickle/unpickle faithfully

4 participants