-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
REF: SparseArray imports #23329
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
REF: SparseArray imports #23329
Conversation
Hello @TomAugspurger! Thanks for submitting the PR.
|
LGTM - thanks for cleaning up the imports using isort too :) |
I think if you remove one of the newlines after |
Codecov Report
@@ Coverage Diff @@
## master #23329 +/- ##
=======================================
Coverage 92.22% 92.22%
=======================================
Files 169 169
Lines 51258 51258
=======================================
Hits 47274 47274
Misses 3984 3984
Continue to review full report at Codecov.
|
Thanks. Merging tonight, since this is a straight refactor, and I have another PR building on top if it. |
pls merge master ping on green. |
CircleCI seems to be ~10 hours behind. OK to cancel those, since we're dropping (most) of them anyway? I'll be free to clean things up this afternoon if we accidentally break anything. |
sure |
arrays/sparse.py was doing an
import pandas as pd
, which is prone to cause circular imports. I've eliminated that import and updated the namespaces appropriately (NaT, isna, algos.unique / factorize).Also updated
setup.cfg
to check the import order inarrays/sparse.py
.