Skip to content

Allow tracklength estimator for neutron heating#3915

Merged
paulromano merged 4 commits into
openmc-dev:developfrom
shimwell:allow-tracklength-estimator-for-heating
May 30, 2026
Merged

Allow tracklength estimator for neutron heating#3915
paulromano merged 4 commits into
openmc-dev:developfrom
shimwell:allow-tracklength-estimator-for-heating

Conversation

@shimwell
Copy link
Copy Markdown
Member

@shimwell shimwell commented Apr 5, 2026

Description

When photon transport is enabled, heating tallies currently get switched to the collision estimator unconditionally.

Is it possible to keep neutron only tallies using tracklength estimator

I think this is also useful alongside #3771 to be able to tally things like heating in silicon from neutrons using the tracklength estimator.

I might be missing some edge cases here as this is not an area of the code I frequent so sorry if I'm way out here.

Fixes # (issue)

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 changed the title Allow tracklength estimator for nutron heating Allow tracklength estimator for neutron heating Apr 6, 2026
Copy link
Copy Markdown
Contributor

@paulromano paulromano left a comment

Choose a reason for hiding this comment

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

Makes sense; thanks @shimwell!

@paulromano paulromano enabled auto-merge (squash) May 30, 2026 04:54
@paulromano paulromano merged commit 219d827 into openmc-dev:develop May 30, 2026
16 checks passed
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.

3 participants