Skip to content

Conversation

jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Nov 18, 2020

  • closes #xxxx
  • tests added / passed
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

discovered on branch that has dataframe always go through split_path

@jreback jreback added the Indexing Related to indexing on series/frames, not to indexes themselves label Nov 18, 2020
@@ -953,6 +953,13 @@ def test_loc_setitem_empty_append(self):
df.loc[:, "x"] = data
tm.assert_frame_equal(df, expected)

# same thing, but with mixed dtypes
Copy link
Contributor

Choose a reason for hiding this comment

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

make a separate test

Copy link
Member Author

Choose a reason for hiding this comment

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

updated+green

@jbrockmendel
Copy link
Member Author

@phofl do you have any other upcoming PRs that affect _setitem_with_indexer_split_path? This and #37931 are my last two before im ready to start in on the always-split-path change, and id like to get any bugfixes+tests in place before that.

@phofl
Copy link
Member

phofl commented Nov 25, 2020

I think only #37749, but will have a closer look. At least nothing which is not already opened

@jreback jreback added the Bug label Nov 26, 2020
@jreback jreback added this to the 1.2 milestone Nov 26, 2020
@jreback
Copy link
Contributor

jreback commented Nov 26, 2020

is this a user facing change? (if so pls add a whatsnew), also merge master and ping on green.

@jbrockmendel
Copy link
Member Author

ping

@jreback jreback merged commit 19c5eea into pandas-dev:master Nov 27, 2020
@jbrockmendel jbrockmendel deleted the bug-indexing-5 branch November 27, 2020 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Indexing Related to indexing on series/frames, not to indexes themselves
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants