Skip to content

Fix integral truediv and floordiv for pyarrow types with large divisor and avoid floating points for floordiv #56677

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.

Sign up for GitHub

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

Merged
merged 25 commits into from
Jan 5, 2024
+104 −14
Merged

Fix integral truediv and floordiv for pyarrow types with large divisor and avoid floating points for floordiv #56677

Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
aad3b2e
avoid floating points for integral floor division
Dec 29, 2023
599420b
comment
Dec 29, 2023
2f43c42
typo
Dec 29, 2023
579d4ca
gh reference
Dec 29, 2023
e826790
improve test
Dec 29, 2023
bfcff0b
fix comment
Dec 29, 2023
cd7c4c7
cleanup
Dec 29, 2023
692dde4
cleanup
Dec 29, 2023
fbae188
whatsnew
Dec 29, 2023
0b3f3c8
remove assert
Dec 29, 2023
2024f4c
revert
Dec 29, 2023
31a8e19
simplify comment
Dec 29, 2023
f7095ba
simplify logic
Dec 29, 2023
80a6976
simplify test
Dec 29, 2023
c992e77
fix uint64 overflow
Dec 29, 2023
e56b1b3
bug fix
Jan 1, 2024
63bf5d8
fix overflow
Jan 1, 2024
167507e
fix comment
Jan 1, 2024
263b8a2
fix truediv for large divsor
Jan 3, 2024
c77c3f7
improve unsigned // unsigned
Jan 3, 2024
e36fb90
improve test cases
Jan 3, 2024
47c4474
negative operand condition
Jan 4, 2024
ae2afa3
Revert "negative operand condition"
Jan 4, 2024
9913c42
fix readability
Jan 4, 2024
51bd7f3
cleanup comments
Jan 5, 2024
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
revert
  • Loading branch information
Rohan Jain committed Dec 29, 2023
commit 2024f4c887d300bd7eaf4631fa974ca4e3a99e65
1 change: 0 additions & 1 deletion doc/source/whatsnew/v2.2.0.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -776,7 +776,6 @@ Timezones
Numeric
^^^^^^^
- Bug in :func:`read_csv` with ``engine="pyarrow"`` causing rounding errors for large integers (:issue:`52505`)
- Bug in :meth:`Series.__floordiv__` for :class:`ArrowDtype` causing rounding errors for large integral results (:issue:`56676`)
- Bug in :meth:`Series.__floordiv__` for :class:`ArrowDtype` with integral dtypes raising for large values (:issue:`56645`)
- Bug in :meth:`Series.pow` not filling missing values correctly (:issue:`55512`)

Expand Down