-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
ENH: __pandas_priority__ #48347
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
ENH: __pandas_priority__ #48347
Conversation
This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this. |
@jbrockmendel I see there is no feedback from others here. But I guess we'll need documentation if we want to merge this. It should also help the discussion, I checked the referred issues, but it's not immediately clear how the new attribute works. |
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.
+1 on this change. I proposed a more detailed version of the documentation, but other than that this looks good, in particular if NumPy is already doing it this way.
@jbrockmendel Would there be a release note needed for this? |
Co-authored-by: Marc Garcia <[email protected]>
…into enh-pandas_priority
Added a whatsnew, not sure how to make a reference to _extending.pandas_priority |
@jreback suggestions on how to get the whatsnew to point to the appropriate place in the docs? |
#@jbrockmendel no idea sorry |
|
Co-authored-by: Matthew Roeschke <[email protected]>
Co-authored-by: Matthew Roeschke <[email protected]>
Co-authored-by: Matthew Roeschke <[email protected]>
green |
Thanks @jbrockmendel |
Will allow dask/modin to address #38946 and modin-project/modin#4646, respectively.
Also can be used to make non-masked EAs defer to masked EAs.