New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DM-36440: Add EpochPropertyMap to track observing epoch in coadds. #724

Merged

erykoff merged 2 commits into main from tickets/DM-36440

Oct 4, 2022

Contributor

erykoff commented Oct 3, 2022

This also fixes the high coverage nside warnings from healsparse which were not fully suppressed.

erykoff added 2 commits

October 3, 2022 11:24


          Add EpochPropertyMap and compute min/mean/max.

eb7b0dc


          Properly suppress warnings about high nside coverage.

cfaefc7

arunkannawadi approved these changes

View reviewed changes

Member

arunkannawadi left a comment

LGTM, and seems to do what we discussed at the Monday Morning Meeting today.

erykoff merged commit 584c84f into main

erykoff deleted the tickets/DM-36440 branch

October 4, 2022 15:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment