-
Notifications
You must be signed in to change notification settings - Fork 509
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 bounding_box property to RectilinearMesh and UnstructuredMesh #2861
Conversation
openmc/mesh.py
Outdated
Axis-aligned cartesian bounding box of cell defined by upper-right and lower- | ||
left coordinates | ||
bounding_box : openmc.BoundingBox | ||
Axis-aligned bounding box of the cell defined by the upper-right and |
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.
Axis-aligned bounding box of the cell defined by the upper-right and | |
Axis-aligned bounding box of the mesh as defined by the upper-right and |
openmc/mesh.py
Outdated
@@ -1666,8 +1675,8 @@ class SphericalMesh(StructuredMesh): | |||
The upper-right corner of the structured mesh. If only two coordinate | |||
are given, it is assumed that the mesh is an x-y mesh. | |||
bounding_box : openmc.BoundingBox | |||
Axis-aligned bounding box of the cell defined by the upper-right and lower- | |||
left coordinates | |||
Axis-aligned bounding box of the cell defined by the upper-right and |
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.
Axis-aligned bounding box of the cell defined by the upper-right and | |
Axis-aligned bounding box of the mesh as defined by the upper-right and |
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 for completing these methods @paulromano! A couple of small nitpicks in the doc strings and we'll get this merged in asap.
Description
I noticed that
bounding_box
,lower_left
, andupper_right
properties were missing on theRectilinearMesh
andUnstructuredMesh
classes. This small PR adds them in. ForUnstructuredMesh
, those properties only work when the mesh has been loaded from a statepoint (and vertex information is available).Checklist
I have run clang-format (version 15) on any C++ source files (if applicable)