Skip to content
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.bartlett: add axis / nan_policy / keepdims support #19392

Merged
merged 2 commits into from Oct 19, 2023

Conversation

mdhaber
Copy link
Contributor

@mdhaber mdhaber commented Oct 15, 2023

Reference issue

Toward gh-14651

What does this implement/fix?

Adds axis / nan_policy / keepdims support to scipy.stats.bartlett.

Additional information

Similar to ansari (gh-19371), for Nd input, bartlett returns a scalar result. However, the result seems to be garbage - it's different from axis=None / raveling the arrays before passing them to the function. So I think it's safe to let axis=0 be the default since there's no useful behavior to maintain.

@mdhaber mdhaber added scipy.stats enhancement A new feature or improvement labels Oct 15, 2023
@mdhaber mdhaber added this to the 1.12.0 milestone Oct 15, 2023
@mdhaber
Copy link
Contributor Author

mdhaber commented Oct 19, 2023

@tirthasheshpatel if this looks good other than the merge conflict, I think it will be safe to resolve it and commit with [skip ci]. Of course, if we need to make other changes, I can merge main before doing that.

Copy link
Member

@tirthasheshpatel tirthasheshpatel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Feel free to just merge main and skip ci.

@mdhaber
Copy link
Contributor Author

mdhaber commented Oct 19, 2023

Oops, when resolving merge conflicts using the web interface, it doesn't allow you to change the commit message : /

@tirthasheshpatel tirthasheshpatel merged commit a5984a4 into scipy:main Oct 19, 2023
13 of 19 checks passed
@tirthasheshpatel
Copy link
Member

No worries, I just merged so the runs should be cancelled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A new feature or improvement scipy.stats
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants