Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add sensitivity analysis for tallies #3179

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Ebiwonjumi
Copy link

@Ebiwonjumi Ebiwonjumi commented Oct 23, 2024

Description

This PR adds tally to score sensitivity coefficients. The tally is useful to perform sensitivity and uncertainty analysis. This feature is based on development by @alhajri on eigenvalue sensitivity analysis.
N.B. Benchmarking the sensitivity tally is still ongoing and still needs lot of work. Folks can take a look to fix it and provide advice.
The feature is applicable to both criticality and fixed-source problems. However, my benchmarking is geared towards fixed-source applications. Usage requires to first run OpenMC to calculate the adjoint flux. Then use the adjoint flux in a second OpenMC run to tally the sensitivity coefficients. Details of how to calculate the adjoint flux in eigenvalue problems and how the sensitivity coefficients are calculated can be found in here

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)

@eepeterson
Copy link
Contributor

Thanks for posting this @Ebiwonjumi. Let's mark it as a draft for now and start the discussion on how to break this down and clean it up a bit.

@Ebiwonjumi Ebiwonjumi marked this pull request as draft October 23, 2024 23:44
@Ebiwonjumi Ebiwonjumi marked this pull request as ready for review October 25, 2024 02:31
@Ebiwonjumi Ebiwonjumi marked this pull request as draft October 25, 2024 16:35
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