BUG: PeriodIndex.to_datetime inconsistent with its docstring #60350
Conversation
| new_data = libperiod.periodarr_to_dt64arr(new_parr.asi8, base) | ||
| is_start = how == "S" | ||
| if is_start: | ||
| new_data = np.asarray( |
There was a problem hiding this comment.
| new_data = np.asarray( | |
| new_data = new_parr.dt.start_time |
the list comprehension is going to be prohibitively slow (might be the cause of the current ASV failures). If you can use the vectorized datetimeindex (DTI) methods we have, I think that might fix things
There was a problem hiding this comment.
Hey thanks @WillAyd, I'll have more time later to look into this, but for now...
new_parr.dt.start_time gives AttributeError: 'PeriodArray' object has no attribute 'dt'
new_parr.start_time gives RecursionError: maximum recursion depth exceeded while calling a Python object
|
This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this. |
|
Thanks for the pull request, but it appears to have gone stale. If interested in continuing, please merge in the main branch, address any review comments and/or failing tests, and we can reopen. |
doc/source/whatsnew/vX.X.X.rstfile if fixing a bug or adding a new feature.