-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: out-of-bounds datetime parsing #15829
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 get the correct behaviour (so raising) on linux-64 (Ubuntu 16.04, python 2 and 3)
|
very weird. The reason I noticed this was #15828 failed on Travis with this & it failed on a local linux-64 vm. (on 2.7) macosx is on 3.6. It possible this is a 2/3 issue. |
I tried both python 3.5 and 2.7, and both worked correctly .. |
@jreback in 0.23.3 I get OutOfBoundsDatetime in both [2] and [3] on both OSX[py27,py37] and Ubuntu16.04[py27,py35]. Do you still get the original behavior? |
Hi, I get that error too on Ubuntu 14.04, Python 3.6.8 Anaconda custom (64-bit):
Any updates on a fix? |
@justmeteomary4 the behavior in [3] you posted is correct; we expect that to raise because pandas Timestamp doesn't support dates before 1677-09-21. If you need to represent an earlier date, use |
This should be fully correct now and could use a test |
For me the test fails for macOS Catalina 10.15.3:
The pandas is a fresh master checkout. Update it appears that the first string, |
I think so, we're going to be following dateutil convention on this. What dateutil do you have installed? |
@jbrockmendel , no update I personally surprised pandas is using its own date/calendar code instead of relying upon some other library. |
After a thought I believe this could be close as "works as expected" unless either there's some clever idea on how to put a warning for this specific case (how specific?), or pandas core team wants to change the way dates are parsed. |
I dont think this is possible. pandas shouldn't be able to install/import without dateutil |
Found it: |
suprisingly [2] and [3] return the same as [1] on linux-64 (meaning they don't raise).
This is from macosx. This is the correct behavior (raising).
The text was updated successfully, but these errors were encountered: