Skip to content

Adding fix and tests for spherical mesh as spatial distribution #3428

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.

Sign up for GitHub

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

Merged
paulromano merged 14 commits into from
Jun 10, 2025
Merged

Adding fix and tests for spherical mesh as spatial distribution #3428

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
6a61d4f
Adding fix and tests for spherical mesh as spatial distribution
pshriwise Jun 4, 2025
e37bbb0
More aptly name from_domain test for spherical mesh
pshriwise Jun 4, 2025
6eb30b5
Clang format
pshriwise Jun 4, 2025
6c2fe30
Further explanation of equal spacing for grids.
pshriwise Jun 4, 2025
06708a2
Reorder phi/theta to match ordering in constructor and other docstrings
pshriwise Jun 4, 2025
e9ffa9d
Making from_domain behavior of spherical and cylindrical meshes consi…
pshriwise Jun 4, 2025
f7cf4af
Make inscribe True the default setting
pshriwise Jun 5, 2025
eaecd62
Removing redundant origin from SphericalMesh.__repr__
pshriwise Jun 5, 2025
575aa70
rm unused import
pshriwise Jun 5, 2025
d7b9fcc
Make inscribed the default
pshriwise Jun 5, 2025
bd813b2
Re-add string return for SphericalMesh.__repr__
pshriwise Jun 6, 2025
66d638d
Update tests/unit_tests/test_source_mesh.py
pshriwise Jun 6, 2025
e12ca21
Switching to enclose_domain
pshriwise Jun 6, 2025
716909a
Introduce HasBoundingBox protocol
paulromano Jun 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removing redundant origin from SphericalMesh.__repr__
  • Loading branch information
@pshriwise
pshriwise committed Jun 5, 2025
commit eaecd62d01d6676864bd1db3b17451f450050596
5 changes: 0 additions & 5 deletions openmc/mesh.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2153,11 +2153,6 @@ def __repr__(self):
if self._phi_grid is not None:
string += fmt.format('\tPhi Min:', '=\t', self._phi_grid[0])
string += fmt.format('\tPhi Max:', '=\t', self._phi_grid[-1])
if self.origin is not None:
string += fmt.format('\tOrigin:', '=\t', self.origin)
else:
string += fmt.format('\tOrigin:', '=\t', '[0.0, 0.0, 0.0]')
return string
Comment thread
pshriwise marked this conversation as resolved.
Resolved

@classmethod
def from_hdf5(cls, group: h5py.Group, mesh_id: int, name: str):
Expand Down