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
Capitalized Boussinesq approximation #1639
Conversation
@@ -2005,13 +2005,13 @@ \subsubsection{Combined formulations} | |||
(Section~\ref{sec:tala}). | |||
|
|||
\item | |||
``boussinesq approximation'': This formulation sets the mass conservation approximation to ``incompressible'', | |||
``Boussinesq approximation'': This formulation sets the mass conservation approximation to ``incompressible'', |
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.
this occurrence of the word needs to match what is being used in the .prm files. Did you change that as well?
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, that should be changed in all files where it occurs (this PR is just a recursive sed find-replace)
include/aspect/parameters.h
Outdated
@@ -123,7 +123,7 @@ namespace aspect | |||
*/ | |||
enum Kind | |||
{ | |||
boussinesq_approximation, | |||
Boussinesq_approximation, |
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 think we should keep it lower-case here, just like we have functions such as solve_stokes()
etc.
include/aspect/parameters.h
Outdated
else if (input == "boussinesq approximation") | ||
return Formulation::boussinesq_approximation; | ||
else if (input == "Boussinesq approximation") | ||
return Formulation::Boussinesq_approximation; |
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.
and this changes what we accept in .prm files
@@ -322,7 +322,7 @@ namespace aspect | |||
prm.enter_subsection("Formulation"); | |||
{ | |||
prm.declare_entry ("Formulation", "custom", | |||
Patterns::Selection ("isothermal compression|custom|anelastic liquid approximation|boussinesq approximation"), | |||
Patterns::Selection ("isothermal compression|custom|anelastic liquid approximation|Boussinesq approximation"), |
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.
this is an incompatible change -- we should discuss
Yes, for this change we should include the sed command to fix it into the |
/run-tests |
Addresses Issue #1623