desihub / desihub/specsim

validate fiberloss fractions against imaging data

Open
#97 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
5
Forks
10
PR merge metrics
No merged PRs in 30d

Description

Looking over #96 (and related code), I don't see where BGS-like targets are handled. Also, it looks like the fiberloss table is computed for just three "pure" astrophysical sources -- point source, EXP, and DEV.

We should do a more careful assessment of these assumptions -- and the resulting fiberloss fractions -- using the (DR7) imaging data.

Digging around some more, perhaps this issue instead belongs in desisim?
https://github.com/desihub/desisim/blob/master/py/desisim/simexp.py#L470-L548

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.