Skip to content

Commit b7b95a8

Browse files
committed
POC: infer time objects to ArrowDtype[time]
1 parent 421166e commit b7b95a8

File tree

3 files changed

+4
-5
lines changed

3 files changed

+4
-5
lines changed

pandas/_libs/lib.pyx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2653,7 +2653,6 @@ def maybe_convert_objects(ndarray[object] objects,
26532653

26542654
elif seen.time_:
26552655
if is_time_array(objects):
2656-
# FIXME: need to ensure this is not timetz
26572656
opt = get_option("future.infer_time")
26582657
if opt is True:
26592658
import pyarrow as pa

pandas/tests/io/test_sql.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2998,16 +2998,15 @@ def test_datetime_date(self):
29982998
def test_datetime_time(self, tz_aware):
29992999
# test support for datetime.time, GH #8341
30003000

3001+
warn_msg = "Pandas type inference with a sequence of `datetime.time`"
30013002
if not tz_aware:
30023003
tz_times = [time(9, 0, 0), time(9, 1, 30)]
3004+
warn = FutureWarning
30033005
else:
30043006
tz_dt = date_range("2013-01-01 09:00:00", periods=2, tz="US/Pacific")
30053007
tz_times = Series(tz_dt.to_pydatetime()).map(lambda dt: dt.timetz())
3008+
warn = None
30063009

3007-
warn_msg = "Pandas type inference with a sequence of `datetime.time`"
3008-
warn = None
3009-
if not tz_aware:
3010-
warn = FutureWarning
30113010
with tm.assert_produces_warning(warn, match=warn_msg):
30123011
df = DataFrame(tz_times, columns=["a"])
30133012

pandas/tests/series/accessors/test_dt_accessor.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ def get_expected(ser, prop):
101101
left = getattr(ser.dt, name)
102102
right = get_expected(ser, name)
103103

104+
104105
if not (is_list_like(left) and is_list_like(right)):
105106
assert left == right
106107
elif isinstance(left, DataFrame):

0 commit comments

Comments
 (0)