Patrick Shriwise

Statepoint file loading refactor and CAPI function

1

# Description This PR contains a slight refactor of the statepoint loading so it can more cleanly be exposed as part of the CAPI. It is also a continuation of...

Motivated by #2873, it's my sense that we'd be more likely to add to the C++ test suite if we select a model to load that covers a variety of...

Color attribute for materials/cells/universes

6

Similar to #2503, I have a plotting query for the community. The assignment of material/cell colors in a dictionary when creating a plotting object has always felt a little clunky...

# Description Partial fix for #2798. This fixes the creation of distribcell filter bin labels for the `tallies.out` file and the `DistribcellFilter::get_all_bins` for models that fill more than one cell...

Set `particle.mu()` at surface crossings.

1

## Description I was talking with @zoeprieto today about tallying angular distributions of surface current by combining a `SurfaceFilter` and `MuFilter`. The `MuFilter` currently relates only to the scattering angle,...

Always write tally spec for writable tallies, even if not active

1

## Description We currently only write tallies to a statepoint file if they're marked as active and writable. Relevant lines filtering out non-writeable, non-active tallies are here https://github.com/openmc-dev/openmc/blob/bd1c2286f8b569bfc863a3da3c67894dcc22df62/src/state_point.cpp#L924-L929 I'm proposing...

Move `test_evaluate_legendre` to the C++ test suite.

1

The `openmc.lib.math.evaluate_legendre` function (and maybe others in that file) exists only for the purpose of testing the C++ implementation of this function. The wrapper can be removed from `openmc/lib/math.py` and...

Based on discussion with @paulromano and @eepeterson: Currently, the `CylindricalMesh` and `SpericalMesh` classes require specification of all grids/parameters upon construction whereas the `RegularMesh` and `RectilinearMesh` classes do not. To better...

Adding methods to automatically apply results to existing Tally objects.

3

# Description One thing I've always found a bit clunky when working with our simulation results is the process of extracting user-specified tally results from the statepoint file. It seems...

# Description The `openmc.stats.Tabular` class assumes that the length of the probabilities provided is equal to the length of the table values. This causes errors when one passes N -1...