-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: minor cleaning of Categorical.__init__ #24402
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
CLN: minor cleaning of Categorical.__init__ #24402
Conversation
Codecov Report
@@ Coverage Diff @@
## master #24402 +/- ##
==========================================
- Coverage 92.3% 92.3% -0.01%
==========================================
Files 162 162
Lines 51875 51871 -4
==========================================
- Hits 47883 47879 -4
Misses 3992 3992
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #24402 +/- ##
==========================================
- Coverage 92.3% 92.3% -0.01%
==========================================
Files 162 163 +1
Lines 51875 51944 +69
==========================================
+ Hits 47883 47946 +63
- Misses 3992 3998 +6
Continue to review full report at Codecov.
|
lgtm. can you run the asv's for categoricals to make sure nothing has changed; we recently had some optimization here which depended on certain checks. |
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.
lgtm pending @jreback's asv comment
I have some problem getting asv working. I don’t think this can have a performance effect now, after the change after @jschendel comments... |
thanks @topper-123 |
At a certain point in the Categorical init, we should for clarity only use
dtype
and not intermixcategories
orordered
.This small PR does that.