Fix bug in MWS FAINT selection

desihub / desitarget

DESI Targeting

BSD 3-Clause "New" or "Revised" License

18 stars 23 forks source link

Fix bug in MWS FAINT selection #733

Closed apcooper closed 3 years ago

apcooper commented 3 years ago

Fix bug in MWS FAINT selection.

apcooper commented 3 years ago

I'm testing this now -- although my understanding of how to run desitarget is a bit out of date. Will report here when I confirm the MWS_FAINT targets are included correctly.

apcooper commented 3 years ago

I confirm the target files produced with this code contain sensible-looking numbers of FAINT_BLUE and FAINT_RED targets.

geordie666 commented 3 years ago

Thanks @apcooper.

I'll merge this, make a new tag, and run new targets in a few hours.