Skip to content

Conversation

@yardasol
Copy link
Owner

@yardasol yardasol commented Jan 6, 2026

Description

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

Fixes # (issue)

Checklist

  • I have performed a self-review of my own code
  • I have run clang-format (version 15) 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)

- parameterize rename_file function
- change base statepoint name used for kinetic simulations (now is
  closer to the base openmc format)
- update corresponding test infrastructure
- add void material flow control to kinetic machinery, in particular the
  calculation of source and scalar flux time derivatives
- add some missing necessary terms to propagation machinery
  (inverse_velocity multiplication)
- add SDP contributions to scalar flux in set_flux_to_flux_plus_source
@yardasol
Copy link
Owner Author

yardasol commented Jan 7, 2026

This shows further improvement in the difference between SDP and TI, probably because now they are truly using the same exact workflow, save for the random ray transport terms themselves

Flux Precursors
rr-vs-pyrk-flux rr-vs-pyrk-precursors
Flux difference Precursors difference
ti-vs-sdp-flux ti-vs-sdp-precursors

@yardasol yardasol force-pushed the kin-rr-sim-simplification branch from 9135531 to 6f17182 Compare January 12, 2026 03:08
Copy link
Owner Author

@yardasol yardasol left a comment

Choose a reason for hiding this comment

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

tests pass :)

@yardasol yardasol merged commit 7aa1a80 into tdrr-clean-refactor Jan 12, 2026
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