search

desihub / desitarget

DESI Targeting
BSD 3-Clause "New" or "Revised" License
18 stars 23 forks source link

STD_WD definition in dark and bright time not consistent between targetmask and mock.build #204

Closed apcooper closed 7 years ago

apcooper commented 7 years ago

Carried over from #169

White dwarfs are being written out as (bright- and dark-time) standards: https://github.com/desihub/desitarget/blob/master/py/desitarget/mock/build.py#L845-L847 but this looks inconsistent with https://github.com/desihub/desitarget/blob/master/py/desitarget/data/targetmask.yaml#L24, where the obsconditions for STD_WD are (DARK|GRAY).

apcooper commented 7 years ago

This is easy to fix in targetmask.yaml (7a69904). The extra bit of logic on STD_WD here is probably unnecessary now?

apcooper commented 7 years ago

Fixed in #208.