Skip to content

REF: Move BaseIndexers and indexer.py functions into their own directory #43065

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 3 commits into from
Aug 18, 2021

Conversation

mroeschke
Copy link
Member

  • Ensure all linting tests pass, see here for how to run them

In preparation for unifying some numba code between groupby and rolling, moving BaseIndexers to a new directory such that

pandas/core/window.indexers.py -> pandas/core/indexers/objects.py (BaseIndexers and their subclasses)
pandas/core/indexers.py -> pandas/core/indexers/utils.py (misc indexer helper functions)

@mroeschke mroeschke added the Refactor Internal refactoring of code label Aug 16, 2021
@mroeschke mroeschke changed the title REF: Move BaseIndexers and indexer.py functions into it's own directory REF: Move BaseIndexers and indexer.py functions into their own directory Aug 16, 2021
@@ -144,7 +144,7 @@
extract_array,
sanitize_array,
)
from pandas.core.indexers import deprecate_ndim_indexing
from pandas.core.indexers.utils import deprecate_ndim_indexing
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you instead import the needed routines in __init__.py and use an __all__ so the original imports will work e.g. from pandas.core.indexer import .....

@jreback jreback added this to the 1.4 milestone Aug 18, 2021
@jreback jreback merged commit 774c2a9 into pandas-dev:master Aug 18, 2021
@mroeschke mroeschke deleted the ref/indexers branch August 19, 2021 00:02
feefladder pushed a commit to feefladder/pandas that referenced this pull request Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor Internal refactoring of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants