-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
API: Deprecate Index.is_(boolean|categorical|integer|floating|numeric|object|interval) #50042
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
Comments
+1 |
take |
I've started working on this, one method at a time. This issue is quite beginner friendly, so If someone wants to join in, they're welcome and I can guide you through it. |
@topper-123 I am interested in this issue. Could you guide m on how to take it? |
hi @ShisuiUzumaki, great. You could for example start with deprecating For a template how to do it, see #50176, where I've deprecated |
I don’t know if the above is sufficient information, but if you’ve got any questions, just get back to me. |
Okay, thanks. I will start working on it and will connect with you if need any guidance. |
This is the first time for joining the issue. I think this will be a good practice for me. Can I have a try? |
I am also looking for a good first issue to work on. Is there anything I can help for this issue? |
take |
great, I see you've started the PRs. Ping me when you're ready for feedback. |
@topper-123 one that will be more difficult (i.e. NOT a good first issue) but that i would find very nice is getting rid of holds_integer. |
@jbrockmendel , yeah I can take that. |
Anyone up for taking on
|
@topper-123 I'm interested in this issue. Can i try to deprecate is_object ? |
Yes! Ping me when the PR is ready for review. |
@ABCPAN-rank, we had a PR for |
@topper-123 yes,i can try it. When index include complex or bool , is_numeric will output flase and is_numeric_dtype will output True . Dose this mean i should change is_numeric_dtype? |
|
Do you mean i can use |
Yes, where it's needed. There may also be places where it's ok to pass complex numbers and booleans. |
Closing as completed. |
Feature Type
Adding new functionality to pandas
Changing existing functionality in pandas
Removing existing functionality in pandas
Problem Description
I propose deprecating the mentioned methods on
Index
.I suggest dtype checking should be the same for indexes, series and arrays, so e.g. if users want to check the dtype of an index they should use the functions in
pandas.api.types
, e.g. for numeric dtype checks, dopd.api.types.is_numeric_dtype(idx)
.Feature Description
Deprecate the mentioned methods and make users use the functions they'd use for series, i.e. the dtype checking functions in
pandas.api.types
.Alternative Solutions
The alternative is not deprecating the methods...
Methods to deprecate:
The text was updated successfully, but these errors were encountered: