-
Notifications
You must be signed in to change notification settings - Fork 90
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
Rename FV::Div_a_Grad_perp to FV::Div_a_Grad_perp #2536
Conversation
This operator is calculating the Divergence of the Gradient (i.e. Laplace), not the Divergence of the Laplacian.
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.
Thanks @dschwoerer I agree the new name is better
438a269
to
1273a26
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.
LGTM, thanks for the nice deprecation notice too!
Co-authored-by: Peter Hill <peter.hill@york.ac.uk>
Failing test is clang-tidy-review which is fixed in both master/next |
Does the title contain |
Oops, sorry, I clearly didn't read things properly! I'll just have to remember to fix it for the changelog |
This operator is calculating the Divergence of the Gradient (i.e.
Laplace), not the Divergence of the Laplacian.
Should this be called
Div_perp_a_Grad_perp
or evenDiv_perp_K_Grad_perp
(similar to the parallel version)?