Closed karllark closed 2 years ago
Still need to remove the old draine_extend code and add docs. And maybe a quick test.
Merging #767 (5add070) into master (bf9ed7a) will increase coverage by
0.29%
. The diff coverage is83.87%
.
@@ Coverage Diff @@
## master #767 +/- ##
==========================================
+ Coverage 42.84% 43.14% +0.29%
==========================================
Files 102 103 +1
Lines 9928 9961 +33
==========================================
+ Hits 4254 4298 +44
+ Misses 5674 5663 -11
Impacted Files | Coverage Δ | |
---|---|---|
beast/physicsmodel/dust/extinction.py | 90.04% <50.00%> (+6.10%) |
:arrow_up: |
beast/physicsmodel/dust/extinction_extension.py | 92.00% <92.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update bf9ed7a...5add070. Read the comment docs.
Plots showing the extended extinction curves compared to the grain models can be seen in the API docs via the RTD build for this PR.
and
Gold star indeed! Thanks, Karl! This looks good to me. Reading the new docs I wonder if there is a/what is the drawback to defaulting to the "extension" models in general (i.e., even without sub-912A observations)?
Hooray. Thanks Karl to make this functionality generic and self-contained eventually! So now the default is for users to indicate whether they want to use an extended version or not in their beast_settings.txt file, right?
Gold star indeed! Thanks, Karl! This looks good to me. Reading the new docs I wonder if there is a/what is the drawback to defaulting to the "extension" models in general (i.e., even without sub-912A observations)?
Yes. The grain model is smoothly merged with the observational model starting at 1675 A. So, the extinction curve in the FUV is changed from what the observations are saying.
Hooray. Thanks Karl to make this functionality generic and self-contained eventually! So now the default is for users to indicate whether they want to use an extended version or not in their beast_settings.txt file, right?
Yes.
I am hoping you can try out this new capability and see if it works as you expected.
Gold star indeed! Thanks, Karl! This looks good to me. Reading the new docs I wonder if there is a/what is the drawback to defaulting to the "extension" models in general (i.e., even without sub-912A observations)?
Yes. The grain model is smoothly merged with the observational model starting at 1675 A. So, the extinction curve in the FUV is changed from what the observations are saying.
I've added a bit more documentation to explain this point.
Gold star = code, tests, and docs. Trying to encourage such PRs. ;-)
Still looking for a formal review of this PR when one of you has the time.
BEAST runs interested in providing predictions of ionizing photons need dust extinction curves that extend below 912 A. This PR provides two new dust extinction models that merge observation based extinction curves with dust grain models to provide this capability.
This is done using the
Generalized_DustExt
capabilities and models available in thedust_extinction
package. Previously this capability was provide directly inside of theGordon16_RvFALaw
using static data files.In the beast_settings.txt file, these new extinction models can be used by declaring the extinction mixture model to be:
extlaw = extinction.Generalized_RvFALaw(ALaw=extinction.Generalized_DustExt(curve=’F19_D03_extension’), BLaw=extinction.Generalized_DustExt(curve=’G03_SMCBar_WD01_extension’))
Replaces #749.