The Wayback Machine - https://web.archive.org/web/20210908070302/https://github.com/pandas-dev/pandas/pull/38335
Skip to content
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: iloc.__setitem__ with dict value and mixed dtypes #38335

Merged
merged 10 commits into from Dec 12, 2020

Conversation

@jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Dec 6, 2020

  • closes #xxxx
  • tests added / passed
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry
expected = df.copy()
rhs = {"x": 9, "y": 99}
df.loc[5] = rhs
expected.loc[5] = [9, 99]
tm.assert_frame_equal(df, expected)

# GH#38335 same thing, mixed dtypes

This comment has been minimized.

@jreback

jreback Dec 8, 2020
Contributor

was this broken too? (or just iloc as indicated)?

is [] ok?

This comment has been minimized.

@jbrockmendel

jbrockmendel Dec 8, 2020
Author Member

loc is broken too. setitem is OK i think.

@jreback jreback added this to the 1.3 milestone Dec 8, 2020
@jreback jreback added the Bug label Dec 8, 2020
pandas/core/indexing.py Show resolved Hide resolved
@jreback jreback merged commit 85acd65 into pandas-dev:master Dec 12, 2020
19 checks passed
19 checks passed
@github-actions
Checks
Details
@github-actions
pre-commit
Details
@github-actions
Web and docs
Details
@codecov
codecov/patch 100.00% of diff hit (target 50.00%)
Details
@codecov
codecov/project 90.63% (target 82.00%)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@azure-pipelines
pandas-dev.pandas Build #20201211.41 succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py37) Linux py37 succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py37_locale_slow) Linux py37_locale_slow succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py37_minimum_versions) Linux py37_minimum_versions succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py37_slow) Linux py37_slow succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py38) Linux py38 succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py38_locale) Linux py38_locale succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py38_np_dev) Linux py38_np_dev succeeded
Details
@azure-pipelines
pandas-dev.pandas (Linux py39) Linux py39 succeeded
Details
@azure-pipelines
pandas-dev.pandas (Windows py37_np16) Windows py37_np16 succeeded
Details
@azure-pipelines
pandas-dev.pandas (Windows py38_np18) Windows py38_np18 succeeded
Details
@azure-pipelines
pandas-dev.pandas (macOS py37_macos) macOS py37_macos succeeded
Details
@azure-pipelines
pandas-dev.pandas (py37_32bit) py37_32bit succeeded
Details
@jbrockmendel jbrockmendel deleted the jbrockmendel:bug-setitem branch Dec 13, 2020
luckyvs1 added a commit to luckyvs1/pandas that referenced this pull request Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants