-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
BUG: parsing pyarrow dtypes corner case #51548
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
Conversation
release note? I think also should now target 2.1 now that rc is cut as this is not a 2.0.x regression or bugfix for a new feature. |
Arrow support is more or less new, I think we can backport relevant fixes there |
wasn't pd.ArrowDtype was added in 1.5? |
Yes, but support was very limited for arrow stuff in 1.5 |
sure. it is experimental and this is not a big change so no objections to targeting 2.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Merged main once more but good for 2.0 IMO
thx @jbrockmendel |
…r case) (#51606) Backport PR #51548: BUG: parsing pyarrow dtypes corner case Co-authored-by: jbrockmendel <[email protected]>
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.pretty dumb corner case