-
Notifications
You must be signed in to change notification settings - Fork 605
Source activity #3795
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
base: develop
Are you sure you want to change the base?
Source activity #3795
Conversation
|
I suggest implementing the activity based timing in the following way:
|
|
I have a few design questions to understand this feature:
This feature is more complicated than I thought. |
|
@GuySten, thanks for your feedback and questions: 1. Does increasing particle number does not mean better statistics but increase of simulated time window? 2. What is the meaning of batches and generations? 3. How do you converge results? 4. Does information about this feature appear in the literature? |
|
I suggest the following changes:
I think in that way we will not have to pre compute decay times and we will get cleaner code. |
Description
This feature adds the
settings.activity_based_timingfeature to OpenMC. When enabled, each source's strength is interpreted as its activity in Bq, and each particle is assigned a birth timestamp sampled from an independent per-source Poisson process.This intentionally reuses
source.strengthrather than introducing a separatesource.activityparameter, since both control "how much" a source contributes (having both leads to ambiguous semantics and redundant logic).The motivation is that, combined with #3785, this gives OpenMC the capability to model coincidence gamma detector simulations: simulate coincident sources with
activity_based_timingand tally heating with a fine TimeFilter in the detector's active volumes.Checklist