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

Add .quantile(foo=quantiles) API? #9805

Open
max-sixty opened this issue Nov 21, 2024 · 1 comment
Open

Add .quantile(foo=quantiles) API? #9805

max-sixty opened this issue Nov 21, 2024 · 1 comment

Comments

@max-sixty
Copy link
Collaborator

Is your feature request related to a problem?

Many of our methods allow the equivalent of .shift(foo=5), where foo is a dimension name.

But not .quantile! That requires .quantile(dim="foo", [0,0.5,1])

Describe the solution you'd like

I guess this is a missing thing rather than a deliberate choice, is that right?

Describe alternatives you've considered

No response

Additional context

No response

@shoyer
Copy link
Member

shoyer commented Nov 22, 2024

I think the API for quantile was adapted from aggregations like mean(), which only need a dimension name.

I agree that this would make sense here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants