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.
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
DM-37411: Add visit-level PSF model robustness metrics #739
Conversation
f5714fb
to
4d33e23
Compare
psfSampling = pexConfig.Field( | ||
dtype=int, | ||
doc="Factor by which to subsample the masked image in each dimension for the maxDistToNearestPsf " | ||
"metric caclulation (the tradeoff is between adequate sampling versus improves speed).", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"improves" -> "improved" (also below), or just drop "improves" entirely.
@@ -69,6 +69,24 @@ class ComputeExposureSummaryStatsConfig(pexConfig.Config): | |||
dtype=str, | |||
default="base_SdssShape_psf" | |||
) | |||
psfSampling = pexConfig.Field( | |||
dtype=int, | |||
doc="Factor by which to subsample the masked image in each dimension for the maxDistToNearestPsf " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Factor by which to subsample" reads to me as the number of points in the grid in each dimension, not the step size. How about "Sampling rate in pixels in each dimension for the maxDistToNearestPsf grid"?
This adds the following two metrics to the computeExposureSummaryStats task: maxDistToNearestPsf: the maximum distance (in pixels) of an unmasked pixel to its nearest PSF star (i.e. star that was used in the PSF modeling). psfTraceRadiusDelta: the delta (in pixels) between the maximum and minimum model PSF trace radius values evaluated on a grid of points lying in the unmasked region of the image. Set to NaN if any of the model PSF trace values evaluate to NaN. These metrics may be used downstream as thresholded criteria for inclusion in coaddition.
Namely, add a threshold on maxPsfTraceRadiusDelta. The default value was conditioned on the HSC-RC2 dataset.
4d33e23
to
edc6b08
Compare
No description provided.