-
Notifications
You must be signed in to change notification settings - Fork 16
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
Update radas to v1 #31
Conversation
46286c6
to
b031741
Compare
38948f4
to
f5411a6
Compare
|
Seems to be slower than the previous version. Why is this? Might want to try evaluate for the entire ne, Te grid at once. Alternatively, pick a single ne value (weak dependence for rates) and just use this single reference value. |
Fixed this in latest commit. Seems to be significantly faster to precompute as much as possible.
|
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.
Walked through this with @tbody-cfs in person. Comments already applied while we reviewed 🎉
LGTM 🚀
Checking the comparison of the Krypton data, was getting an issue where the Lz curve was going to zero at a particular temperature. Have now added logic to drop invalid rows and columns from the atomic data. Raw data
New output of interpolator, dropping invalid values |
Removes the radas submodule and replaces it with a pip-installed version.