Skip to content

Add specification for computing a tensor contraction (linalg: tensordot) #136

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

Merged
merged 5 commits into from
May 12, 2021

Conversation

kgryte
Copy link
Contributor

@kgryte kgryte commented Mar 8, 2021

This PR

  • specifies the interface for performing a tensor contraction.
  • is derived from comparing signatures across array libraries.

Notes

  • JAX supports providing a precision keyword argument to control matrix-multiplication precision. It is alone in doing so among compared array libraries.

  • TF allows providing a (2,K) shaped array for the axes keyword argument. NumPy et al support array-like values or lists when specifying multiple axes. This proposal follows NumPy et al in only allowing int and Sequence.

  • TF returns an array whose data type is that of x1. This proposal requires that the result follow type promotion rules.

@rgommers rgommers added the API extension Adds new functions or objects to the API. label Mar 20, 2021
@rgommers rgommers force-pushed the main branch 3 times, most recently from 0607525 to 138e963 Compare April 19, 2021 20:25
@kgryte
Copy link
Contributor Author

kgryte commented May 12, 2021

This PR has been open for some time without comment and discussed/approved during meetings. Will merge and we can submit follow-up PRs should any issues/concerns arise.

@kgryte kgryte merged commit 0768fc8 into main May 12, 2021
@kgryte kgryte deleted the tensordot branch May 12, 2021 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API extension Adds new functions or objects to the API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants