Further updates to the LRG selection for SV3.

desihub / desitarget

DESI Targeting

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

18 stars 23 forks source link

Further updates to the LRG selection for SV3. #703

Closed geordie666 closed 3 years ago

geordie666 commented 3 years ago

This PR focuses on additional updates to the LRG targets in preparation for the 1% Survey. It:

Changes the bright-end cut for LRG targets from zfibertot > 16 to zfibertot > 17.5.
Adds a low-density (600-per-sq.-deg.) LRG sample (denoted LRG_LOWDENS).
- The LRG_LOWDENS bit is purely informational as the LRG_LOWDENS targets are a strict subset of the LRG targets.
Also updates the intersphinx URLs to fix the online documentation builds.

geordie666 commented 3 years ago

@rongpu: It would be great if you could quickly check that I didn't mess anything up for the SV3 LRGs in this PR. You can focus your review on sv3_cuts.py as there are some peripheral changes in this PR related to bit-masks and documentation builds.

rongpu commented 3 years ago

@geordie666 I have checked the changes in sv3_cuts.py and they look good. Thanks!

geordie666 commented 3 years ago

Thanks @rongpu. I'll merge this now.