Skip to content

Conversation

@phofl
Copy link
Member

@phofl phofl commented Nov 1, 2025

@phofl phofl added Resample resample method Non-Nano datetime64/timedelta64 with non-nanosecond resolution labels Nov 1, 2025
@phofl phofl added this to the 3.0 milestone Nov 2, 2025
pandas_datetime_to_datetimestruct(val, creso, &dts)
else:
# Plain datetime/date
pydate_to_dtstruct(stamp, &dts)
Copy link
Member

Choose a reason for hiding this comment

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

should this by pydatetime_to_dtstruct?

Copy link
Member Author

Choose a reason for hiding this comment

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

I wondered about this too but we seem to be using the date version a little earlier in the callstsack as well

I think we actually never get here with a real datetime

Copy link
Member

Choose a reason for hiding this comment

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

i guess datetime subclasses date, so this should be safe.

@jbrockmendel
Copy link
Member

small comment, otherwise LGTM

@jbrockmendel jbrockmendel merged commit ffc6fee into pandas-dev:main Nov 3, 2025
52 checks passed
@jbrockmendel
Copy link
Member

thanks @phofl

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

Labels

Non-Nano datetime64/timedelta64 with non-nanosecond resolution Resample resample method

2 participants