-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: the __from_arrow__ conversion for numeric arrays broken if buffer size doesn't match #40896
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
Labels
Milestone
Comments
The issue comes from those lines: pandas/pandas/core/arrays/_arrow_utils.py Lines 28 to 29 in 4aeb8f2
Where we need to slice the buffer from |
I'd be happy to look into this! |
Related: JDASoftwareGroup/kartothek#410 |
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 19, 2021
Add Arrow buffer slicing before handing it over to numpy which is needed in case the Arrow buffer contains padding or offset.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 19, 2021
Add Arrow buffer slicing before handing it over to numpy which is needed in case the Arrow buffer contains padding or offset.
4 tasks
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 19, 2021
Use numpy dtype for bitwidth information. Fix test signature.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 20, 2021
Fix mask buffer in test.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 20, 2021
Move tests to pandas/tests/arrays/masked/test_arrow_compat.py Add more dtypes to test.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 20, 2021
Change argument passed to pyarrow_array_to_numpy_and_mask() from str to np.dtype.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 21, 2021
Modify test_arrow_compat.py: - Use import_optional_dependency to skip the whole module if pyarrow is not available. - Use any_real_dtype fixture.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
No changes to the code, only to one docstring. Test if CI passes when started at a different time of the day.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
Add whatsnew entry.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
Enforce creation of Arrow bitmask buffer.
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
Merge branch 'master' into issue-40896
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
ThomasBlauthQC
added a commit
to ThomasBlauthQC/pandas
that referenced
this issue
Apr 22, 2021
jorisvandenbossche
pushed a commit
that referenced
this issue
Apr 28, 2021
yeshsurya
pushed a commit
to yeshsurya/pandas
that referenced
this issue
May 6, 2021
JulianWgs
pushed a commit
to JulianWgs/pandas
that referenced
this issue
Jul 3, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Report from https://issues.apache.org/jira/browse/ARROW-12336
The original reproducer is (but it would be good to see if we can find a simpler case for a test):
which gives
The text was updated successfully, but these errors were encountered: