Skip to content

Turn on weight windows if weight_windows_file is specified#4007

Merged
GuySten merged 1 commit into
openmc-dev:developfrom
paulromano:turn-on-wws
Jul 9, 2026
Merged

Turn on weight windows if weight_windows_file is specified#4007
GuySten merged 1 commit into
openmc-dev:developfrom
paulromano:turn-on-wws

Conversation

@paulromano

Copy link
Copy Markdown
Contributor

Description

When a user specifies settings.weight_windows, OpenMC automatically will turn weight windows on. However, when they specify settings.weight_windows_file, it doesn't automatically turn weight windows on. This PR changes the behavior so that weight windows are automatically enabled if settings.weight_windows_file is specified.

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 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.

Looks good to me.

@GuySten
GuySten enabled auto-merge (squash) July 9, 2026 23:13
@paulromano paulromano mentioned this pull request Jul 9, 2026
5 tasks
@GuySten
GuySten merged commit 3c3ebba into openmc-dev:develop Jul 9, 2026
16 checks passed
@paulromano
paulromano deleted the turn-on-wws branch July 10, 2026 04:18
@shimwell shimwell added the Merging Soon PR will be merged in < 24 hrs if no further comments are made. label Jul 10, 2026
jtramm pushed a commit to jtramm/openmc that referenced this pull request Jul 17, 2026
jtramm pushed a commit to jtramm/openmc that referenced this pull request Jul 17, 2026
With the auto-enable behavior cherry-picked from develop (openmc-dev#4007),
convert_to_multigroup and the bootstrap test no longer need to set
weight_windows_on explicitly alongside the file.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merging Soon PR will be merged in < 24 hrs if no further comments are made.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants