search

desihub / desiutil

General DESI utilities, shell scripts, desiInstall, etc.
BSD 3-Clause "New" or "Revised" License
3 stars 9 forks source link

add brick_tan_wcs_size() function: compute required size of tiles #177

Closed dstndstn closed 3 years ago

dstndstn commented 3 years ago

... in order to make overlapping TAN-projection WCS tiles on brick centers

For example, a Bricks object with 0.25-degree spacing requires TAN WCS tiles at least 0.26445 degrees across to guarantee that tiles on Bricks centers will cover the entire sky without holes.

weaverba137 commented 3 years ago

This looks OK so far, but be sure to update the change log before I do a formal review.

coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.3%) to 66.903% when pulling fd62a3f2f8f029d620262a95d04de26e479eb176 on add-tan-size into 129dc445f3f51c25815bd53e4995fbc87d35da11 on master.

weaverba137 commented 3 years ago

Any further changes planned?

dstndstn commented 3 years ago

Thanks for your fixes. I didn't have anything further planned.