-
-
Notifications
You must be signed in to change notification settings - Fork 5.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
Add minmax and extrema #5275
Add minmax and extrema #5275
Conversation
I would like this to be merged soon if there's no big issues. |
+1 |
Nice! About the names, though: while it makes sense when you have understood the Or maybe just drop |
Here, |
What's the use case for |
|
Good enough for me. I'd say we should go ahead and merge this soon. |
Ah, I missed the fact that |
One of the key reason to motivate the use of You may think about |
There was a very long debate when we did the The general principle that we finally settled on is to use different function names for element-wise operations and reduction. |
Yeah. The difference here is that |
Semantics:
Please let me know if there is any problem. I will add entries to helpdb & news.md after this is merged.