-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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: stats.describe: add array API support #20667
Conversation
CI failure appears to be unrelated. |
Thanks @mdhaber planning to review this and other from Wednesday onwards |
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.
One minor comment then this is good go
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.
Thanks @mdhaber
Reference issue
Towards #20544
What does this implement/fix?
Adds array API support to
scipy.stats.describe
.Additional information
I think our policy of translating existing tests rather than rewriting is good, but it was pretty cumbersome for this one. Most of what we really need is one big property-based test showing that the results of
describe
match the results of the functions it calls. Those functions are tested elsewhere.