-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
REGR: fix DataFrame.diff with read-only data #35707
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
REGR: fix DataFrame.diff with read-only data #35707
Conversation
doc/source/whatsnew/v1.1.1.rst
Outdated
@@ -20,6 +20,7 @@ Fixed regressions | |||
- Fixed regression where :func:`pandas.testing.assert_series_equal` would raise an error when non-numeric dtypes were passed with ``check_exact=True`` (:issue:`35446`) | |||
- Fixed regression in :class:`pandas.core.groupby.RollingGroupby` where column selection was ignored (:issue:`35486`) | |||
- Fixed regression in :meth:`DataFrame.shift` with ``axis=1`` and heterogeneous dtypes (:issue:`35488`) | |||
- Fixed regression in :meth:`DataFrame.diff` with read-only data (:issue:`35559`). |
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.
i think no period at the end, otherwise LGTM
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.
we are likely inconsistent about this generally, but fixed here.
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.
lgtm. i am not sure i would actually leave the commented code, or if you do add a specific tag: TODO (or TODO-CYTHON or something)
@jreback that's what I already did ( |
ahh right, ok then. |
Thanks @jorisvandenbossche |
@meeseeksdev backport 1.1.x |
) Co-authored-by: Joris Van den Bossche <[email protected]>
Closes #35559