-
Notifications
You must be signed in to change notification settings - Fork 2
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
Updates Serpent models #25
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, it looks good. Nice consistency in cell and material numbering between files. A few questions about things that weren't obvious to me.
serpent/compact
Outdated
* Kernel radius 2.125e-6 cm * | ||
* Buffer radius 3.125e-6 cm * | ||
* IPyC radius 3.475e-6 cm * | ||
* SiC radius 3.825e-6 cm * | ||
* OPyC radius 4.225e-6 cm * |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these just unit conversions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. This is just a comment though, it doesn't affect the simulation
"wo16: 0.0897065977520648\n", | ||
"wc12: 0.0224266494380162\n", | ||
"\n", | ||
"Nu235: 0.0036633202990892723\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What exactly are the N values? The number density?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
serpent/compact
Outdated
* Kernel radius 212.5e-6 m * | ||
* Buffer radius 312.5e-6 m * | ||
* IPyC radius 347.5e-6 m * | ||
* SiC radius 382.5e-6 m * | ||
* OPyC radius 422.5e-6 m * | ||
* Kernel radius 2.125e-6 cm * | ||
* Buffer radius 3.125e-6 cm * | ||
* IPyC radius 3.475e-6 cm * | ||
* SiC radius 3.825e-6 cm * | ||
* OPyC radius 4.225e-6 cm * |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these just unit conversions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. This is just a comment though, it doesn't affect the simulation
@@ -17,7 +17,6 @@ | |||
* OPyC radius 422.5e-6 m * | |||
* Matrix Graphite * |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you also want to reflect the unit conversions here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, I just fixed all the unit conversions. They were all wrong ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks for making the changes. I'll go ahead and merge this PR.
Fixes #24, #26, and #27.
This PR additionally defines an input file for a fuel column with lumped burnable poisons.
And it also adds a jupyter notebook with the material composition calculation.