Skip to content

Conversation

@jbrockmendel
Copy link
Member

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

ATM we cast strings for listlike inputs but not for scalars.

@jreback jreback added Indexing Related to indexing on series/frames, not to indexes themselves Timezones Timezone data dtype labels Sep 12, 2020
@jreback jreback added this to the 1.2 milestone Sep 12, 2020
@jreback
Copy link
Contributor

jreback commented Sep 12, 2020

I guess this is technically a bug fix on DTA, maybe add a note; ping on green.

@jbrockmendel
Copy link
Member Author

ping

@jreback jreback merged commit 229722e into pandas-dev:master Sep 12, 2020
@jbrockmendel
Copy link
Member Author

thoughts on if we want to support this in Series.setitem etc? that's pretty easy, just define DatetimeLikeBlock._can_hold_element to wrap DatetimeLikeArray._validate_setitem_value xref #36226

@jbrockmendel jbrockmendel deleted the unify-validators-cast_str branch September 12, 2020 23:41
kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
…#36261)

* ENH: consistently cast strings for DTA/TDA/PA.__setitem__

* whatsnew
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Indexing Related to indexing on series/frames, not to indexes themselves Timezones Timezone data dtype

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants