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

More configuraton options in fissionSource #101

Merged

Conversation

Mikolaj-A-Kowalski
Copy link
Collaborator

Resolves #96.
Add few options and makes cosmetic changes to the fissionSource

  • Changes the default number of re-sample limit from 200 to 10000
  • Allows user to change the allowed number of re-sample attempts
  • Allows user to specify AABB where to try to place the points.
  • Add User Manual entry for the fission source that seems to be missing.

Allows user to specify maximum number of resamples and spatial bounds
where to place the particles.
Co-authored-by: valeriaRaffuzzi <108435337+valeriaRaffuzzi@users.noreply.github.com>
Copy link
Member

@valeriaRaffuzzi valeriaRaffuzzi left a comment

Choose a reason for hiding this comment

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

Nice, always good to have extra options :)

@Mikolaj-A-Kowalski Mikolaj-A-Kowalski merged commit 93169c3 into CambridgeNuclear:main Nov 21, 2023
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.

Initialise problem
2 participants