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
Trilinos parallel is_non_negative() function #12968
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.
Looks good - I have two small suggestions to make w.r.t. the MPI reduction.
58994a6
to
4f70017
Compare
/rebuild |
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 to me, but @juleoc02 is my student and when possible I try to avoid merging my students' PRs. So I would appreciate someone else do it!
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 to me. The implementation is pretty much the same as for all_zero
.
suggested changes have been implemented :-)
Changed the is_non_negative() function to work with distributed vectors in Trilinos. Mostly copied from the above all_zero() function that already worked in parallel