-
-
Notifications
You must be signed in to change notification settings - Fork 18.2k
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
EA interface: rename ExtensionArray._hasnans to ._hasna #45519
Conversation
@jorisvandenbossche no more on 1.4 |
This is an innocent name change, but important to do now since it's public API. Given that the release is planned tomorrow or Sunday, I think that is certainly still enough time to merge something like this. |
I chose _hasnans to be consistent with the Index.hasnans method, dont really give a darn. |
we could change that too |
As that's public wouldn't that need a deprecation |
no its experimental on an EA and private on Index |
This comment has been minimized.
This comment has been minimized.
@meeseeksdev backport 1.4.x |
@jorisvandenbossche if you can followup with a change in the docs (e.g. add _hasna) & also we need to followup on Index. |
This comment has been minimized.
This comment has been minimized.
this needs a manual backport @jorisvandenbossche @simonjayhawkins |
ok. should probably remove from the docs https://pandas.pydata.org/pandas-docs/dev/reference/api/pandas.Index.hasnans.html |
…snans to ._hasna
@@ -841,7 +841,7 @@ def _isnan(self) -> npt.NDArray[np.bool_]: | |||
return self.asi8 == iNaT | |||
|
|||
@property # NB: override with cache_readonly in immutable subclasses |
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.
the comment here should be updated, might no longer be relevant
|
@simonjayhawkins thanks for the backport! |
As discussed in follow-up on #45024 (comment)
Since this is "public" for developers, we need to decide on a name now for 1.4. Using "na" instead of "nan" makes it consistent with the other missing values related methods in the EA interface (isna, dropna, fillna, _can_hold_na)