-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: Frame.iat item_cache invalidation bug #45706
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
jbrockmendel
commented
Jan 30, 2022
•
edited
Loading
edited
- closes BUG: Label and integer based indexing return different values for same column #45684
- tests added / passed
- Ensure all linting tests pass, see here for how to run them
- 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.
Could you test for iloc too? Same case as in op had the wrong behavior too.
also I think this is a regressio, should go into 1.4.1
doc/source/whatsnew/v1.5.0.rst
Outdated
@@ -268,6 +268,7 @@ Indexing | |||
- Bug in :meth:`Series.__setitem__` where setting :attr:`NA` into a numeric-dtpye :class:`Series` would incorrectly upcast to object-dtype rather than treating the value as ``np.nan`` (:issue:`44199`) | |||
- Bug in :meth:`DataFrame.mask` with ``inplace=True`` and ``ExtensionDtype`` columns incorrectly raising (:issue:`45577`) | |||
- Bug in getting a column from a DataFrame with an object-dtype row index with datetime-like values: the resulting Series now preserves the exact object-dtype Index from the parent DataFrame (:issue:`42950`) | |||
- Bug in :meth:`DataFrame.iat` setting values leading to not propagating correctly in subsequent lookups (:issue:`45684`) |
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.
move to 1.4.1
updated + green |
@meeseeksdev backport 1.4.x |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@jbrockmendel if you can push the manual backport here |