Skip to content
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

bug fix for xsGroup manager w/ coupling #1118

Merged
merged 3 commits into from Jan 25, 2023
Merged

Conversation

albeanth
Copy link
Member

@albeanth albeanth commented Jan 24, 2023

Description

Bug fix to create consistency with latticePhysicsInterface::interactCoupled for tight coupling.

This change to the lattice physics interface was snuck into PR #1067 and limited the lattice physics interface to execute only on time node 0 of each cycle. This resulted in an inconsistency between the cross section group manager and the lattice physics interface. This PR serves to resolve this inconsistency.


Checklist

  • This PR has only one purpose or idea.
  • Tests have been added/updated to verify that the new/changed code works.
  • The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
  • The documentation is still up-to-date in the doc folder.
  • The dependencies are still up-to-date in setup.py.

@albeanth albeanth added the bug Something is wrong: Highest Priority label Jan 24, 2023
@albeanth albeanth marked this pull request as ready for review January 24, 2023 22:29
@albeanth albeanth merged commit 2a95ca6 into terrapower:main Jan 25, 2023
@albeanth albeanth deleted the xsGroupsFix branch January 25, 2023 18:41
drewj-usnctech added a commit to drewj-usnctech/armi that referenced this pull request Jan 25, 2023
…30/assembly-1d-parent-class

* terrapower/main:
  Updating B4C docstring for assumed TD frac (terrapower#1117)
  bug fix for xsGroup manager w/ coupling (terrapower#1118)
  Add neutronics category to linPow parameter (terrapower#1119)
  Refactoring Material to not subclass Composite (terrapower#1062)
  Exposing tight coupling to `OperatorSnapshots` (terrapower#1113)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong: Highest Priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants