Skip to content

Override NJOY default damage energy threshold#4022

Merged
paulromano merged 2 commits into
openmc-dev:developfrom
GuySten:damage-threshold
Jul 20, 2026
Merged

Override NJOY default damage energy threshold#4022
paulromano merged 2 commits into
openmc-dev:developfrom
GuySten:damage-threshold

Conversation

@GuySten

@GuySten GuySten commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Description

Please include a summary of the change and which issue is fixed if applicable. Please also include relevant motivation and context.

Currently when generating neutron data with NJOY we use the default NJOY displacement threshold energy. (NJOY has different defaults for various atoms).
This value is used to calculate the damage-energy score.
This PR let us override the NJOY defaults with our own threshold energy.

Checklist

  • I have performed a self-review of my own code
  • I have run clang-format (version 18) on any C++ source files (if applicable)
  • I have followed the style guidelines for Python source files (if applicable)
  • I have made corresponding changes to the documentation (if applicable)
  • I have added tests that prove my fix is effective or that my feature works (if applicable)

@GuySten
GuySten requested a review from paulromano as a code owner July 19, 2026 14:13
@GuySten GuySten changed the title control NJOY damage energy threshold Override NJOY default damage energy threshold Jul 19, 2026

@paulromano paulromano left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @GuySten. I changed the name to displacement_energy to align with standard terminology.

@paulromano
paulromano enabled auto-merge (squash) July 20, 2026 14:08
@paulromano
paulromano merged commit 61c8a59 into openmc-dev:develop Jul 20, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants