Skip to content

Add a length multiplier option for XDG meshes. - #4155

Merged
paulromano merged 2 commits into
openmc-dev:developfrom
nuclearkevin:add_len_mul
Oct 3, 2026
Merged

paulromano merged 2 commits into
openmc-dev:developfrom
nuclearkevin:add_len_mul

Conversation

@nuclearkevin

Copy link
Copy Markdown
Member

Description

I've been working on integrating XDG mesh tallies into Cardinal, and found that the "external XDG" option was missing an optional parameter for length scaling (which we use to match OpenMC models in cm to MOOSE models in m).

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)

@nuclearkevin
nuclearkevin requested a review from pshriwise October 2, 2026 19:03

@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 @nuclearkevin!

@paulromano
paulromano merged commit 0c2b484 into openmc-dev:develop Oct 3, 2026
16 checks passed
@nuclearkevin
nuclearkevin deleted the add_len_mul branch October 3, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants