Skip to content
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

interpolate stellar templates #315

Open
moustakas opened this issue Jan 23, 2018 · 5 comments
Open

interpolate stellar templates #315

moustakas opened this issue Jan 23, 2018 · 5 comments

Comments

@moustakas
Copy link
Member

When passed an input metadata table with "real" TEFF, LOGG, and FEH values for stars (or just TEFF and LOGG in the case of white dwarfs), the stellar templates generated by templates.STAR and templates.WD should -- but do not -- interpolate the templates. This issue can lead to the kind of gridding seen in desihub/desitarget#251 and elsewhere, and may also be affecting the flux-calibration portion of the pipeline.

This issue has been worked on and discussed in many places, e.g.:

Recently, @callendeprieto wrote the gettemplate module to call his long-used and well-tested ferre code to do the interpolation. However, using ferre as-is would introduce a dependence on Fortran.

Let's track developments and progress here.

@julienguy
Copy link
Contributor

I think we should not interpolate the standard star templates in the simulations.
Interpolations introduce errors. We would loose the ability to test the accuracy of the fit of standard stars at a level better than the interpolation errors.

@sbailey
Copy link
Contributor

sbailey commented Jan 25, 2018

Whether or not we interpolate, the true spectrum is kept in the simspec file and can/should be used for comparisons rather than going back to the original template files. Although flux calibration might use a different interpolation scheme (or no interpolation), and both might not be completely physically realistic, allowing them to be different is more realistic than keeping both the input and the template fitting on exactly the same grid.

@callendeprieto
Copy link

callendeprieto commented Jan 25, 2018 via email

@moustakas
Copy link
Member Author

I think it's worth leaving this issue open for now, but I'll note that in #434 we have implemented a much finer grid of stellar templates, which should help mitigate the need for interpolation (which we don't do).

@julienguy
Copy link
Contributor

A note that we have to verify the flux calibration is still working fine with the new stellar templates v3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants