desihub / desisim

DESI simulations
BSD 3-Clause "New" or "Revised" License
16 stars 22 forks source link

write some code to deal with the DB white dwarf models #200

Closed moustakas closed 7 years ago

moustakas commented 7 years ago

The white dwarf models now include both DB and DA white dwarfs (as indicated by the WDTYPE tag in the metadata table). Need to write some code to generate the requested white dwarf type. See also https://github.com/desihub/desisim-testdata/pull/6.

sbailey commented 7 years ago

For consideration: the redshift fitting outputs include a "SUBTYPE" column. Its exact usage remains to be defined, but it is a placeholder for things like stellar classifications wanting to record a subtype while QSO classifications might want to record a subtype from a different set of options.

Should/could template metadata do the same? — a catch-all SUBTYPE column whose options depend upon what type of template it is, rather than carrying a WDTYPE column that is meaningful for only a small fraction of the overall targets.

moustakas commented 7 years ago

Implementing this would require a somewhat annoying regeneration of the metadata tables (at least for white dwarfs and stars). Could we just do this on-the-fly as needed when the template-generating code gets called, e.g., in newexp-desi? We do something like this (i.e., picking and choosing what we want from the template metadata tables and renaming some columns) anyway...

moustakas commented 7 years ago

Addressed in #213.