-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Cannot create DatetimeIndex using Period #6780
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.
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
Comments
I think the real issue is that |
yes it could work keeping in mind that it's defaulted to a start of the period it could be that's why it's not automatic |
Yeah, I'm not sure what's best there actually. I just ran into another issue comparing datetime to period doesn't work, but it's not clear to me how it should actually work. End vs beginning of period defaults for Period are bound to be controversial. Maybe it should just raise though maybe with a more informative error message? I don't like it (more work for me), but our code really should be more careful here and there's no real reason to assume beginning vs end of period mapping. |
actually this does report a |
Yeah, I was just thinking something like "Cannot convert Period to Timestamp unambiguously. Use to_timestamp" |
sounds good to me |
closed by #8405 |
This doesn't work. Looks like
start
andend
should try a bit harder to convert to timestamp?This does.
The text was updated successfully, but these errors were encountered: