Remove self loops from spontaneous fission decay mode#3907
Remove self loops from spontaneous fission decay mode#3907paulromano merged 4 commits intoopenmc-dev:developfrom
Conversation
|
Skipping the gain term for sf in As a possible follow-up (separate PR), it might be worth also fixing this upstream in |
|
I agree, but if you wanted to add spontaneous fission yields to actually populate the correct gain terms you'd still need a specific check for |
paulromano
left a comment
There was a problem hiding this comment.
Thanks @eepeterson for the PR and @shimwell for the further suggestion. I've fixed the underlying issue here, which is that DecayMode.daughter was returning the parent nuclide for the 'sf' decay mode; now it returns None.
Description
This PR just prevents spontaneous fission decay modes from erroneously adding source terms along the diagonal of the burnup matrix. Since the official depletion chains have this behavior, this seems like the easiest immediate fix, but it does warrant a discussion on whether the semantics should be changed in the chain file themselves.
Fixes #3906
Checklist