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
Add second version of TensorProductMatrixSymmetricSum::apply_inverse() #14178
Conversation
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.
Otherwise looks good
/** | ||
* Same as above but the user can provide a user-owned temporal array, | ||
* resolving the two issues described above. | ||
*/ |
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.
It looks like the temporary array needs to be of length 2 * n
: could you document that here?
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.
The array is set to the right size internally.
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.
Got it. Could we document that instead? We typically require that users set output arrays to the right sizes so we should clarify what to do for work arrays.
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.
I have added a comment!
34f1ddb
to
2300115
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.
Looks good, but there are a few typos in the text.
2300115
to
8e3533f
Compare
@kronbichler Updated! |
We run into timeouts for some tests affected by this:
Can you take a look? |
8e3533f
to
baef48f
Compare
@kronbichler Should be fixed now. I din't move the mutex but duplicated it. So that the code dead locked. |
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.
Good.
Add second version of TensorProductMatrixSymmetricSum::apply_inverse()
No description provided.