-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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
TST: sql add more tests (NaN handling, ..) #7100
TST: sql add more tests (NaN handling, ..) #7100
Conversation
ok....maybe an issue specifically to track the fixing of these (and sub-link to the issues with checkboxes?) and if you want to push some to 0.14.1 ok too |
Added it to the issue #2754 |
TST: sql add more tests (NaN handling, ..)
these are currently breaking on windows because the should it be? its having a parse error on the NaT |
|
Strange, I tested this on Windows (python 2.7) and it works. Something with the python version? |
its happenin on several version 26-64,33-32,33-64 (where I have things installed) |
I can't reproduce this on windows, python 2.7, 64bit, sqlalchemy 0.8.2, so difficult to debug. But I suppose something is different in the datetime parsing between different versions, leading to an error or to NaN. |
yes |
- tests were failing on some combination of platform/version, see #7100, so skip for now until real solution is implemented
This PR just adds some tests, not fixes to get them all running.
Summary:
0001-255-255T00:00:00
.