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-37884: Include all evaluateMeanPsfFwhm in try/except blocks #249
Conversation
python/lsst/ip/diffim/makeKernel.py
Outdated
except InvalidParameterError: | ||
self.log.info("Unable to evaluate PSF at the average position. " | ||
"Evaluting PSF on a grid of points." | ||
) |
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.
I don't know if logging this information in many places is helpful. For a given template with a CoaddPsf
error, this line might end up appearing several times.
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.
I think you're right. This message would appear once already from subtractImages.py
, so I would suggest downgrading the log messages here (and below) to debug
fb935de
to
603d04d
Compare
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.
Looks good. I would suggest these logs be downgraded to debug
to reduce the clutter.
python/lsst/ip/diffim/makeKernel.py
Outdated
except InvalidParameterError: | ||
self.log.info("Unable to evaluate PSF at the average position. " | ||
"Evaluting PSF on a grid of points." | ||
) |
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.
I think you're right. This message would appear once already from subtractImages.py
, so I would suggest downgrading the log messages here (and below) to debug
python/lsst/ip/diffim/makeKernel.py
Outdated
templateFwhmPix = getPsfFwhm(template.psf) | ||
scienceFwhmPix = getPsfFwhm(science.psf) | ||
except InvalidParameterError: | ||
self.log.info("Unable to evaluate PSF at the average position. " |
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.
As above, downgrade to debug
Thanks for the comment, Ian. I downgraded the logging level for these to be |
f9a22ad
to
7f3cbdd
Compare
7f3cbdd
to
091ee38
Compare
This PR introduces the fix that should have been in place in #246 . All evaluateMeanPsfFwhm calls must be within an
except
block and should be called only aftergetPsfFwhm
has been tried first.