-
Notifications
You must be signed in to change notification settings - Fork 8
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
Weedout #281
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #281 +/- ##
==========================================
- Coverage 90.19% 89.95% -0.25%
==========================================
Files 26 27 +1
Lines 2070 2130 +60
==========================================
+ Hits 1867 1916 +49
- Misses 203 214 +11 ☔ View full report in Codecov by Sentry. |
OK, the finished version of this filters lines by looking at the ratio of the line-center alpha (estimated assuming a tophat profile with width equal to the Doppler width) to the continuum alpha. This seems to work well. Following that, the lines that make it through are ordered by EW. This is so that you can run There is also a function Two general changes to the code made as part of this PR:
|
This PR implements a function to identify all lines below a certain depth, were they synthesized in isolation. It is an attempt to address #278.
This implementation takes a first pass, identifying important lines by comparing their wavelength-integrated absorption coefficient to the total absorption, then does single-line synthesis on a more limited set of lines in the linelist. This works well for line depths down to 1% (and possible shallower), but it's not very fast. It's probably sufficient for now.
TODO