-
-
Notifications
You must be signed in to change notification settings - Fork 421
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
Continuum Opacities #1917
Continuum Opacities #1917
Conversation
Before a pull request is accepted, it must meet the following criteria:
|
Codecov Report
@@ Coverage Diff @@
## master #1917 +/- ##
==========================================
- Coverage 59.04% 58.76% -0.28%
==========================================
Files 70 70
Lines 7767 7834 +67
==========================================
+ Hits 4586 4604 +18
- Misses 3181 3230 +49
Continue to review full report at Codecov.
|
…he opacities.py file. Removed Continuum class and put relevant arrays into the numba plasma. Updated tests accordingly. Co-authored-by: sonachitchyan chitchyan.sona@gmail.com Co-authored-by: chvogl cvogl@mpa-garching.mpg.de
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Fixed slowdown caused by previous merge of the continuum branch. Continuum processes can now be enabled.
WARNING: tests still required for running with continuum processes!
TODO: remove old plasma opacity calculators
Moved the continuum opacity calculations over to opacities.py
How has this been tested?
Examples
Type of change
Checklist