Ilham Variansyah
Ilham Variansyah
Hi @LiZhulun. Thank you for pointing a bug! If you don't mind, can you please send me (email: [email protected]) your input files? Also @paulromano and @nelsonag, I'm so sorry for...
Hi @nelsonag, I'm so sorry for the very slow progress; I still want to work on this PR! I'm targetting to get some progress going starting next month. Is there...
Hi, @paulromano and @nelsonag I rebased, resolved conflicts, and force-pushed the branch. (I hope it is done correctly...) I applied @nelsonag suggestion on using MGXS-inverse_speed instead of mg_speeds. It's interesting...
I was trying to write a regression_test for the feature and found an issue: Originally, OpenMC uses the running average of k as the keff for the simulation in the...
Hi @paulromano and @nelsonag, I think the PR is ready for review. # Summary It has been a while since the PR was first opened. So, here is the summary:...
@gridley I would be happy to take this on! Should I make a PR for it? Sjenitzer's "Dynamic MC" can be seen as an orchestra of several methods, and those...
The additional step of sampling delayed emission time shifts the random number sequence used in the rest of the random walk, which would make some (if not most) of the...
> If we're tracking particle times during normal simulation, the delayed emission times should not be incorrect, IMO. Correctness should not be an option that's off by default--this was the...
There was a little bug in the use of MG_mode + collision estimator + survival_biasing: https://github.com/openmc-dev/openmc/pull/2898/commits/6ca1ae09547dd78684b31a1bfe155d6eed141b2b @gridley @nelsonag (Note: I had to use the collision estimator due to the current...
No problem, @paulromano. Thanks! I think that is the only change needed in the random number stream for analog time-dependent simulations. Unless there are other reactions that produce delayed particles...