-
Notifications
You must be signed in to change notification settings - Fork 7
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
added neutronics tools #23
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.
Thanks @SteSeg for taking care of the neutronics part!
-
The CI is failing here because it's missing the
h5py
package. It is also missingopenmc
as a dependency. Could you add it to the requirements?Line 18 in eef7af6
dependencies = ["numpy", "pint", "scipy", "matplotlib", "sympy", "pandas"] -
Moreover you will have to add it to the ci workflow file?
-
Also the _version.py file should not be tracked. Did you do
git add .
? 😈 -
Can we add a test to check that this function works?
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.
The CI is still not passing due to openmc not being installed with conda. I can take care of this if you want!
Got a few other comments
# angular and energy distribution | ||
|
||
from pathlib import Path | ||
from collections.abc import Iterable |
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.
Probably not important but do you know if there's a difference between this and
from typing import List
# angle column names | ||
angles = df.columns[1:] | ||
# angular bins in [0, pi) | ||
pbins = np.cos([np.deg2rad(float(a)) for a in angles] + [np.pi]) |
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.
pretty sure there's a way to use angles
directly without iterating through it and converting it to a float. pandas
builds upon numpy
Co-authored-by: Rémi Delaporte-Mathurin <[email protected]>
Co-authored-by: Rémi Delaporte-Mathurin <[email protected]>
Co-authored-by: Rémi Delaporte-Mathurin <[email protected]>
Co-authored-by: Rémi Delaporte-Mathurin <[email protected]>
Co-authored-by: Rémi Delaporte-Mathurin <[email protected]>
Co-authored-by: Rémi Delaporte-Mathurin <[email protected]>
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.
CI passes 🎉
Added the neutronics folder in the package with the neutron generator characteristics and a function to generate it as openmc source.