-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
DOC/TST: Indexing with NA raises #30308
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
492f904
DOC/TST: Indexing with NA raises
TomAugspurger 6444aa0
Merge remote-tracking branch 'upstream/master' into na-indexing-raises
TomAugspurger 53f4f63
Handle BooleanArray in all EAs
TomAugspurger 3bbf868
update
TomAugspurger a5ac457
fixups
TomAugspurger 0dfe761
type
TomAugspurger dac111d
fix benchmark
TomAugspurger d1f08d9
fixup
TomAugspurger 3dd59ca
typo
TomAugspurger 151bdfe
updates
TomAugspurger d57b0ac
Revert "updates"
TomAugspurger 36be0f6
examples
TomAugspurger 7bd6c2f
restore datetime fix
TomAugspurger c5f3afb
Merge remote-tracking branch 'upstream/master' into na-indexing-raises
TomAugspurger 76bb6ce
Merge branch 'master' of https://github.com/pandas-dev/pandas into na…
TomAugspurger 505112e
update error message
TomAugspurger c73ae8e
checks
TomAugspurger 3efe359
Merge remote-tracking branch 'upstream/master' into na-indexing-raises
TomAugspurger f94483f
update for error message
TomAugspurger 953938d
Merge remote-tracking branch 'upstream/master' into na-indexing-raises
TomAugspurger 8b1e567
update isort
TomAugspurger f317c64
isort
TomAugspurger c656292
fixup
TomAugspurger d4f0adc
Merge branch 'master' of https://github.com/pandas-dev/pandas into na…
TomAugspurger 37ea95e
fixup
TomAugspurger 816a47c
Merge remote-tracking branch 'upstream/master' into na-indexing-raises
TomAugspurger 21fd589
update arrayo
TomAugspurger 3637070
doc
TomAugspurger 61599f2
integer
TomAugspurger 6a0eda6
Merge remote-tracking branch 'upstream/master' into na-indexing-raises
TomAugspurger e622826
fixup
TomAugspurger 5004d91
fixup
TomAugspurger 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
update
- Loading branch information
commit 3bbf868eb1033b9aace90921cef4d8a460ecfa85
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
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.
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.
is there a real reason to expose check_book_array_indexer now?
is_bool_indexer also shouldn't be in api.extension, api.types would be ok, though it was never meant to be public.
can we defer both of these as exposing to the public until we actually see use?
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.
Happy to defer exposing those utilities. Will push an update this evening.