-
Notifications
You must be signed in to change notification settings - Fork 4.3k
GH-40282: [Python] Use C++ type traits #40761
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
Closed
Closed
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
df521f3
change is_nested() to use C++ type traits
llama90 1daac63
apply lint
llama90 5c67223
update to maintain compatibility
llama90 0f60359
update type_traits to expose all type check functions provided
llama90 818a7ab
update to code readability
llama90 085f67c
update docstring
llama90 edc1f9b
apply lint
llama90 38750b7
delete not used types
llama90 cbaada0
update by self-review
llama90 4d18b7e
enhance docstring to help users understand how to use the function
llama90 3d38c7e
update is_integer to check bit width
llama90 837e153
fix return type for bit_width and offset_bit_width
llama90 b1d9b10
update to support the logic usign the provided functions
llama90 8810b4b
update docstring removing 'Check if~' and 'Useful for'
llama90 cec4b11
update docstring removing (...)
llama90 e9a4954
update function to start with _ prefix
llama90 974df0c
Update python/pyarrow/type_traits.pxi
llama90 30adfec
apply lint
llama90 8f08f46
delete type_traits.pxi
llama90 153de4b
migrate functions to types.pxi and expose through types.py
llama90 cbd5c2e
apply lint
llama90 06f1f1e
update by self-review
llama90 ed78341
add docstring to avoid below error
llama90 c583351
update test_types to avoid pa.types.
llama90 ac1d5f9
unexport ensure_type function
llama90 4ba8af0
update docstring
llama90 9b2e795
update extern statements
llama90 3bbc1e2
fix code for test_numpy_array_protocol
llama90 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
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.