-
-
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
replace size(A, d1, d2, ds...)
with size(A, (dims...))
#26851
Comments
Ah, so you'd want to still support Note that the |
Yes, the proposal would be that these three forms are supported:
|
Or we could just deprecate it entirely... |
Supporting a tuple argument would be a bigger change. Many This was only used within tests of |
Right now the general signature
size(A, dims...)
does different things depending on whether there are zero, one or many dimensions provided. It seems like it would be better to separate this into: no dimension argument, scalar dimension argument, and tuple dimension argument.The text was updated successfully, but these errors were encountered: