-
-
Notifications
You must be signed in to change notification settings - Fork 11.9k
API: Add Array API aliases (math, bitwise, linalg, misc) [Array API] #25086
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
Conversation
78e94b2 to
4a1c7e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a language tweak for the release note
|
Looks good to me. Since this has only been open for two days and it's adding new things to the public API I'm going to wait until next week to merge this. In the meantime if anyone has any issues with adding these new aliases please let us know. |
1c0f23e to
9e04686
Compare
4a97c44 to
f4be2ee
Compare
84a5b5a to
c6c5c7a
Compare
|
This one has conflicts too! Sorry about not merging this in timely manner like I said I would a month ago. |
3236f0f to
7b077e2
Compare
|
I rebased it, now CI is green! |
Co-authored-by: Pieter Eendebak <pieter.eendebak@gmail.com> Co-authored-by: Nathan Goldbaum <nathan.goldbaum@gmail.com>
7b077e2 to
0c3cad6
Compare
|
OK, let's pull this one in next. Thanks @mtsokol! |
Hi @rgommers @ngoldbaum,
This PR adds Array API compatible functions that are direct aliases of existing functions:
acos, acosh, asin, asinh, atan, atanh, atan2bitwise_left_shift, bitwise_invert, bitwise_right_shifttensordot, matmulconcat, permute_dims, powTracking issue: #25076