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
Remove template parameter from FPArrayComparator #14294
Conversation
Tensor<1, | ||
FPArrayComparator<VectorizedArrayType>::width, | ||
typename FPArrayComparator<VectorizedArrayType>::Number>> &t1, |
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.
Can't you just say
Tensor<1, | |
FPArrayComparator<VectorizedArrayType>::width, | |
typename FPArrayComparator<VectorizedArrayType>::Number>> &t1, | |
Tensor<1, width, Number>> &t1, |
here and below, since this is the definition of a member function?
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.
Yes this worked 👍
a22baf0
to
541d777
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.
Nice. Much better now with the short names.
I have added the "Do not merge" label since we first need to agree on #14293. |
541d777
to
a7c0d74
Compare
/rebuild |
a7c0d74
to
94a807d
Compare
... with the intention that it not only works for
VectorizedArray<Number>
but also forNumber
.In forllow up PRs, I plan
Table
sTensorProductMatrixSymmetricSumCache
depends on #14293