-
Notifications
You must be signed in to change notification settings - Fork 66
Allow controlling number of particles with a postprocessor #1239
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
Conversation
|
Job Documentation, step Sync to remote on ddc8214 wanted to post the following: View the site here This comment will be updated on new commits. |
test/tests/neutronics/heat_source/particles_from_postprocessor.i
Outdated
Show resolved
Hide resolved
test/tests/neutronics/heat_source/particles_from_postprocessor.i
Outdated
Show resolved
Hide resolved
e9a0301 to
7a392c2
Compare
test/tests/neutronics/heat_source/particles_from_postprocessor.i
Outdated
Show resolved
Hide resolved
test/tests/neutronics/heat_source/particles_from_postprocessor.i
Outdated
Show resolved
Hide resolved
test/tests/neutronics/heat_source/particles_from_postprocessor.i
Outdated
Show resolved
Hide resolved
|
Nice addition! This is clean and relatively unintrusive to the code - just some minor style & testing comments from me. |
8c4037b to
8de6a22
Compare
|
The VTB test failure is unrelated |
|
PR to fix the VTB failure is up. VTB tests should start passing one this is merged. |
|
Job Test VTB on ab4fc4f : invalidated by @meltawila |
ab4fc4f to
53a0421
Compare
…wski relaxation is used
Co-authored-by: April Novak <novak@berkeley.edu>
53a0421 to
7e62e7c
Compare
|
thanks @meltawila this looks good after these last comments LGTM! |
Co-authored-by: April Novak <novak@berkeley.edu>
|
Job Precheck, step Clang format on c5e7bc4 wanted to post the following: Your code requires style changes. A patch was auto generated and copied here
Alternatively, with your repository up to date and in the top level of your repository:
|
…icles() as the accessor
closes #1237