-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: frame.loc[2:, 'z'] not setting inplace when multi-block #44345
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
Nov 7, 2021
- closes #xxxx
- tests added / passed
- Ensure all linting tests pass, see here for how to run them
- whatsnew entry
bf072b0
to
8dd52be
Compare
@@ -407,6 +407,11 @@ def test_setitem_frame_duplicate_columns(self, using_array_manager): | |||
expected["C"] = expected["C"].astype("int64") | |||
# TODO(ArrayManager) .loc still overwrites | |||
expected["B"] = expected["B"].astype("int64") | |||
|
|||
mark = pytest.mark.xfail( |
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.
what this about?
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.
the ArrayManager behavior is different for reasons i'm still chasing down
is this user facing at all? |
yes, will update with whatsnew |
whatsnew added + green |
cc @phofl if you can have a look |
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