-
Notifications
You must be signed in to change notification settings - Fork 191
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
Introduce AMR policies #5765
Introduce AMR policies #5765
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.
Doc CI failure is #5764.
Isotropic /**< all dimensions must be refined the same */ | ||
}; | ||
|
||
/// Output operator for . |
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.
"Isotropy"
#include <pup_stl.h> | ||
|
||
namespace amr { | ||
Policies::Policies(amr::Isotropy isotropy) : isotropy_(isotropy) {} |
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.
const argument.
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.
Initially only holds the policy for the isotropy. Additional policies will be added in future PRs.
This gets rid of an unnecessary if constexpr as well as not needing to add the tag in executables.
Proposed changes
Adds
amr::Policies
, a class that will allow run-time control over adaptive mesh refinement features.This PR adds the ability to control whether refinement is isotropic or anisotropic (i.e. whether each dimension can refine independently).
Upgrade instructions
Code review checklist
make doc
to generate the documentation locally intoBUILD_DIR/docs/html
.Then open
index.html
.code review guide.
bugfix
ornew feature
if appropriate.Further comments