-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
BUG: Fix unpickling of string dtypes of legacy pandas versions #61770
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
BUG: Fix unpickling of string dtypes of legacy pandas versions #61770
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Liam3851 thanks a lot for the bug report and the fix!
Looks perfect, and thanks for adding legacy data (we should probably also add some data for 2.0-2.2 ..).
Can you add a note in the doc/source/whatsnew/v2.3.1.rst
file? Because we will want to backport this fix
Thanks very much for the review @jorisvandenbossche, I've added pickles for 2.0-2.2 as extra checks and a whatsnew entry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
… legacy pandas versions
@@ -59,6 +59,7 @@ Bug fixes | |||
- Bug in :meth:`.DataFrameGroupBy.min`, :meth:`.DataFrameGroupBy.max`, :meth:`.Resampler.min`, :meth:`.Resampler.max` where all NA values of string dtype would return float instead of string dtype (:issue:`60810`) | |||
- Bug in :meth:`DataFrame.sum` with ``axis=1``, :meth:`.DataFrameGroupBy.sum` or :meth:`.SeriesGroupBy.sum` with ``skipna=True``, and :meth:`.Resampler.sum` with all NA values of :class:`StringDtype` resulted in ``0`` instead of the empty string ``""`` (:issue:`60229`) | |||
- Fixed bug in :meth:`DataFrame.explode` and :meth:`Series.explode` where methods would fail with ``dtype="str"`` (:issue:`61623`) | |||
- Fixed bug in unpickling objects pickled in pandas versions pre-2.3.0 that used :class:`StringDtype` (:issue:`61763`). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Liam3851 for the PR.
For future reference, by convention the trailing period is normally excluded. But no need to do anything as a follow up as will probably be changed when the release notes are tidied just prior to release.
doc/source/whatsnew/v2.3.1.rst
file if fixing a bug or adding a new feature.