Skip to content

Commit

Permalink
Update armi/materials/tests/test_fluids.py
Browse files Browse the repository at this point in the history
  • Loading branch information
john-science authored Feb 6, 2025
1 parent 760f9d2 commit a363726
Showing 1 changed file with 8 additions and 9 deletions.
17 changes: 8 additions & 9 deletions armi/materials/tests/test_fluids.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,15 +36,14 @@ def test_fluidDensityWrapperNoWarning(self):
"""Test that Component.material.density does not raise a warning for fluids.
The ARMI Framework contains a mechanism to warn users if they ask for the density of a
material attached to a component, e.g., ``comp.material.density()``. The problem is the
component is the source of truth for volume and composition. It stores dimensions, which can
be thermally expanded according to the material and number densities, which may be updated
during operation. Much of the framework operates on ``Component.density`` and other
``Component`` methods for mass accounting. However, ``comp.material.density`` does not know
about the new composition or volumes and can diverge from ``component.density``.
Additionally, the framework does not do any thermal expansion on fluids. So the above calls to
``component.material.density`` are warranted if the material is a fluid.
material attached to a component. But the component is the source of truth for volume and
composition. And can be thermally expanded during operation. Much of the framework operates
on ``Component.density`` and other ``Component`` methods for mass accounting. However,
``comp.material.density`` does not know about the new composition or volumes and can diverge
from ``component.density``.
Additionally, the framework does not do any thermal expansion on fluids. So the above calls
to ``component.material.density`` are warranted for fluids.
"""
self._checkCompDensityLogs(
mat=self.MySolid(),
Expand Down

0 comments on commit a363726

Please sign in to comment.