-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
BUG: read_csv with dtype=bool[pyarrow] #53391
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
doc/source/whatsnew/v2.0.2.rst
Outdated
@@ -31,6 +31,7 @@ Bug fixes | |||
- Bug in :func:`api.interchange.from_dataframe` was unnecessarily raising on bitmasks (:issue:`49888`) | |||
- Bug in :func:`merge` when merging on datetime columns on different resolutions (:issue:`53200`) | |||
- Bug in :func:`read_csv` raising ``OverflowError`` for ``engine="pyarrow"`` and ``parse_dates`` set (:issue:`53295`) | |||
- Bug in :func:`read_csv` when defining ``dtype`` with ``bool[pyarrow]`` (:issue:`53390`) |
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.
This is only for "c"/"python" parsers, right?
Might want to clarify that here.
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.
Yeah correct, updated.
False | ||
True | ||
""" | ||
result = parser.read_csv(StringIO(data), dtype={"col": "bool[pyarrow]"}) |
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.
Can you add a test for dtype_backend=pyarrow
as well?
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.
Sure, I modified test_EA_types
to test this instead
Ah sorry, I forgot the 2.0.2 release was today. Can you move the whatsnew note to 2.0.3? |
Owee, I'm MrMeeseeks, Look at me. There seem to be a conflict, please backport manually. Here are approximate instructions:
And apply the correct labels and milestones. Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon! Remember to remove the If these instructions are inaccurate, feel free to suggest an improvement. |
* Backport PR #53391: BUG: read_csv with dtype=bool[pyarrow] * Add xfail
* BUG: read_csv with dtype=bool[pyarrow] * Use existing test instead * Clarify whatsnew * Move to 2.0.3
* BUG: read_csv with dtype=bool[pyarrow] * Use existing test instead * Clarify whatsnew * Move to 2.0.3
dtype
is defined withbool[pyarrow]
#53390 (Replace xxxx with the GitHub issue number)doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.