-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
Consider supporting multiple axes in reduce #7915
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
Comments
NumPy does support multiple axes in reduce. You just need to specify them with a tuple, not a list:
|
Oh, sorry! What's the argument for disallowing other collections? |
Also, regardless, the error message ("an integer is required") should probably be fixed. |
I don't think there's a principled reasons for disallowing other sequences for the axis argument. The logic for parsing the axis argument is written in C, though, so duck typing may be a little trickier (in any case it was not done). #7916 will fix the error message. |
Got it, thanks. |
@shoyer I think there is a desire to keep things simple and consistent. The topic could be revisited on the mailing list if we want to make a formal rule. Indeed, maybe we should have some formal rules about the common keywords. |
Tensorflow supports multiple reduction axes. Numpy does not. Is this a useful feature to have?
The text was updated successfully, but these errors were encountered: