8000 BUG: Issues with DatetimeTZ values in where and combine_first (#21469 + #21546) by Liam3851 · Pull Request #21660 · pandas-dev/pandas · GitHub
[go: up one dir, main page]
Skip to content

BUG: Issues with DatetimeTZ values in where and combine_first (#21469 + #21546) #21660

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
Jul 2, 2018
Merged

BUG: Issues with DatetimeTZ values in where and combine_first (#21469 + #21546) #21660

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
ac2f836
Minimal modifications that pass new tests and fix #21469 + #21546.
Liam3851 Jun 27, 2018
770a22e
Remove unused import.
Liam3851 Jun 27, 2018
04a0538
Remove explicit DatetimeTZBlock check.
Liam3851 Jun 27, 2018
6e6c619
Update test_coercion; xfailing case now fixed.
Liam3851 Jun 27, 2018
8c022d9
Remove orphaned private _where_compat method.
Liam3851 Jun 27, 2018
96a59f6
Remove newly-unused import
Liam3851 Jun 28, 2018
deda82c
Merge remote-tracking branch 'upstream/master' into where_tz
Liam3851 Jun 28, 2018
71a4fd4
Remove no-longer-relevant comment and no-op code line.
Liam3851 Jun 29, 2018
7c14643
Clean up unit tests.
Liam3851 Jun 29, 2018
3d072c5
Fix unneeded imports.
Liam3851 Jun 29, 2018
f812e78
Add whatsnew.
Liam3851 Jun 29, 2018
f177ec5
Merge remote-tracking branch 'upstream/master' into where_tz
Liam3851 Jun 29, 2018
09fe18f
Merge branch 'master' into PR_TOOL_MERGE_PR_21660
jreback Jul 2, 2018
c97b951
doc
jreback Jul 2, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
doc
  • Loading branch information
@jreback
jreback committed Jul 2, 2018
commit c97b951f434e02cc98d986ac7c1a8016fb7ca37f
12 changes: 8 additions & 4 deletions doc/source/whatsnew/v0.24.0.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,6 @@ Timezones
- Bug in :meth:`Series.truncate` with a tz-aware :class:`DatetimeIndex` which would cause a core dump (:issue:`9243`)
- Bug in :class:`Series` constructor which would coerce tz-aware and tz-naive :class:`Timestamp`s to tz-aware (:issue:`13051`)
- Bug in :class:`Index` with ``datetime64[ns, tz]`` dtype that did not localize integer data correctly (:issue:`20964`)
- Bug in :meth:`Series.combine_first` with ``datetime64[ns, tz]`` dtype which would return tz-naive result (:issue:`21469`)
- Bug in :class:`DatetimeIndex` where constructing with an integer and tz would not localize correctly (:issue:`12619`)

Offsets
Expand Down Expand Up @@ -288,15 +287,18 @@ Indexing
^^^^^^^^

- The traceback from a ``KeyError`` when asking ``.loc`` for a single missing label is now shorter and more clear (:issue:`21557`)
- When ``.ix`` is asked for a missing integer label in a :class:`MultiIndex` with a first level of integer type, it now raises a ``KeyError`` - consistently with the case of a flat :class:`Int64Index` - rather than falling back to positional indexing (:issue:`21593`)
- When ``.ix`` is asked for a missing integer label in a :class:`MultiIndex` with a first level of integer type, it now raises a ``KeyError``, consistently with the case of a flat :class:`Int64Index, rather than falling back to positional indexing (:issue:`21593`)
- Bug in :meth:`DatetimeIndex.reindex` when reindexing a tz-naive and tz-aware :class:`DatetimeIndex` (:issue:`8306`)
- Bug in :class:`DataFrame` when setting values with ``.loc`` and a timezone aware :class:`DatetimeIndex` (:issue:`11365`)
- Bug when indexing :class:`DatetimeIndex` with nanosecond resolution dates and timezones (:issue:`11679`)
- Bug in :meth:`Series.where` and :meth:`DataFrame.where` with ``datetime64[ns, tz]`` dtype (:issue:`21546`)
- Bug in :func:`DataFrame.fillna` where a ``ValueError`` would raise when one column contained a ``datetime64[ns, tz]`` dtype (:issue:`15522`)

-

Missing
^^^^^^^

- Bug in :func:`DataFrame.fillna` where a ``ValueError`` would raise when one column contained a ``datetime64[ns, tz]`` dtype (:issue:`15522`)

MultiIndex
^^^^^^^^^^

Expand Down Expand Up @@ -337,6 +339,8 @@ Reshaping
^^^^^^^^^

- Bug in :func:`pandas.concat` when joining resampled DataFrames with timezone aware index (:issue:`13783`)
- Bug in :meth:`Series.combine_first` with ``datetime64[ns, tz]`` dtype which would return tz-naive result (:issue:`21469`)
- Bug in :meth:`Series.where` and :meth:`DataFrame.where` with ``datetime64[ns, tz]`` dtype (:issue:`21546`)
-
-

Expand Down
0