Review of DC2_Coadd_Run1.1p_Butler_Access_HSC_selection.ipynb

LSSTDESC / DC2-production

Configuration, production, validation specifications and tools for the DC2 Data Set.

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

11 stars 7 forks source link

Review of DC2_Coadd_Run1.1p_Butler_Access_HSC_selection.ipynb #208

Closed sschmidt23 closed 6 years ago

sschmidt23 commented 6 years ago

Overall very clear and well documented

Very minor comments:

Expand all acronyms, even if it seems repetitive, for those just getting started who may not be familiar (DRP, HSC, etc...)

skymap = butler.get('deepCoadd_skyMap') could use a little more explanation in the preceding comment, say a bit about what it stores about the tracts and patches as it's a little vague currently.

sschmidt23 commented 6 years ago

This is addressed in #213 so I think it can be closed.