-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: pd._libs.lib.is_iterator reports Timestamp as iterable on Cygwin #45158
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
On OSX I get this on master
Could you show a full traceback of the build logs? |
I'm not sure what you mean by "full traceback from build logs." The full build log is bigger than the maximum 65536-character limit for a comment, and I can't figure out how to attach. |
You could pipe the build log to a text file and attach the file to a Github comment |
build.log |
I got a Cygwin compile running with GitHub Actions; this looks like one instance where this error arises. You should be able to see the build log from earlier on that same page. |
I updated the Cygwin compile to merge the latest What information do you still need? |
Thanks for confirming @DWesl. |
I just checked a few other cases, and it looks like |
Fixes pandas-dev#45158 Keep original implementation if not on Cygwin, and choose the implementation at compile time to avoid performance hits from the extra branch. [Test results on Cygwin are available are available here](https://github.com/DWesl/pandas/runs/5648361139?check_suite_focus=true)
Given #46477 (comment), it appears that this is an issue that should be fixed upstream. Closing |
Pandas version checks
I have checked that this issue has not already been reported.
I have confirmed this bug exists on the latest version of pandas.
I have confirmed this bug exists on the master branch of pandas.
Reproducible Example
Issue Description
I have noticed two functions in pandas that use the above pattern:
This is causing tests to fail as I try to build 1.3.5. Both the attempt shown above and the attempt with 1.3.5 are on Cygwin. The problem does not occur with pandas 1.0.5, 1.1.2, or 1.3.5 on Windows. All attempts were made with the CPython implementation.
Expected Behavior
I would expect either
is_iterator
to returnFalse
orlist
to succeed.Installed Versions
The text was updated successfully, but these errors were encountered: