desihub/speclite

do the filter curves really have the atmosphere included?

Closed this issue · 5 comments

@dkirkby The filters purport to have the atmosphere included
https://speclite.readthedocs.io/en/latest/filters.html
but, e.g., the SDSS i-band filter appears to be missing the telluric bands around ~7500 A, e.g.
https://www.sdss.org/instruments/camera/#Filters

Any thoughts?

Good catch @moustakas! I just did a quick check against the tabulated source data and you are right that the atmosphere is not included:
sdss

(shaded = w/o atmos., curve = w/ X=1.3 atmos.)

Before I fix this, let me just check that X=1.3 is what you actually want?

I'm not certain. Let me ask on-list.

Actually, this issue just affects the SDSS filters, right? So let's just stick with X=1.3. Thanks.

@moustakas, does this issue need to remain open?

Done in #76.