From ebd8e7fd485007e02576edc393d5681b0f216b88 Mon Sep 17 00:00:00 2001 From: Arun Kannawadi Date: Sun, 19 Oct 2025 16:57:26 -0400 Subject: [PATCH 1/3] Update the JIRA link in docs --- doc/lsst.pipe.tasks/index.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/lsst.pipe.tasks/index.rst b/doc/lsst.pipe.tasks/index.rst index d9f394700..883743eb5 100644 --- a/doc/lsst.pipe.tasks/index.rst +++ b/doc/lsst.pipe.tasks/index.rst @@ -29,7 +29,7 @@ Contributing ============ ``lsst.pipe.tasks`` is developed at https://github.com/lsst/pipe_tasks. -You can find Jira issues for this module under the `pipe_tasks `_ component. +You can find Jira issues for this module under the `pipe_tasks `_ component. .. If there are topics related to developing this module (rather than using it), link to this from a toctree placed here. From 1e97c53177469203b879f941dd5a5025998e200a Mon Sep 17 00:00:00 2001 From: Arun Kannawadi Date: Sun, 19 Oct 2025 17:00:59 -0400 Subject: [PATCH 2/3] Reduce logging verbosity in InterpImageTask --- python/lsst/pipe/tasks/interpImage.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/python/lsst/pipe/tasks/interpImage.py b/python/lsst/pipe/tasks/interpImage.py index 1b59c0e9a..b865e57f5 100644 --- a/python/lsst/pipe/tasks/interpImage.py +++ b/python/lsst/pipe/tasks/interpImage.py @@ -122,8 +122,8 @@ def _setFallbackValue(self, mi=None): "negativeFallbackAllowed is False: setting fallbackValue to 0.0") fallbackValue = max(fallbackValue, 0.0) - self.log.info("fallbackValueType %s has been set to %.4f", - self.config.fallbackValueType, fallbackValue) + self.log.debug("fallbackValueType %s has been set to %.4f", + self.config.fallbackValueType, fallbackValue) return fallbackValue @@ -184,11 +184,11 @@ def run(self, image, planeName=None, fwhmPixels=None, defects=None): # set psf from exposure if provided OR using modelPsf with fwhmPixels provided try: psf = image.getPsf() - self.log.info("Setting psf for interpolation from image") + self.log.trace("Setting psf for interpolation from image") except AttributeError: - self.log.info("Creating psf model for interpolation from fwhm(pixels) = %s", - str(fwhmPixels) if fwhmPixels is not None else - (str(self.config.modelPsf.defaultFwhm)) + " [default]") + self.log.debug("Creating psf model for interpolation from fwhm(pixels) = %s", + str(fwhmPixels) if fwhmPixels is not None else + (str(self.config.modelPsf.defaultFwhm)) + " [default]") psf = self.config.modelPsf.apply(fwhm=fwhmPixels) fallbackValue = 0.0 # interpolateOverDefects needs this to be a float, regardless if it is used @@ -197,7 +197,7 @@ def run(self, image, planeName=None, fwhmPixels=None, defects=None): self.interpolateImage(maskedImage, psf, defectList, fallbackValue) - self.log.info("Interpolated over %d %s pixels.", len(defectList), planeName) + self.log.debug("Interpolated over %d %s pixels.", len(defectList), planeName) self.metadata["interpolated_pixel_count"] = len(defectList) @contextmanager From 44ef1b676a3ec5ca1d7c7cd5ceb7597d10afbb36 Mon Sep 17 00:00:00 2001 From: Arun Kannawadi Date: Mon, 20 Oct 2025 20:37:20 -0400 Subject: [PATCH 3/3] Avoid unnecessary type casting --- python/lsst/pipe/tasks/interpImage.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/lsst/pipe/tasks/interpImage.py b/python/lsst/pipe/tasks/interpImage.py index b865e57f5..6259c7d41 100644 --- a/python/lsst/pipe/tasks/interpImage.py +++ b/python/lsst/pipe/tasks/interpImage.py @@ -187,7 +187,7 @@ def run(self, image, planeName=None, fwhmPixels=None, defects=None): self.log.trace("Setting psf for interpolation from image") except AttributeError: self.log.debug("Creating psf model for interpolation from fwhm(pixels) = %s", - str(fwhmPixels) if fwhmPixels is not None else + fwhmPixels if fwhmPixels is not None else (str(self.config.modelPsf.defaultFwhm)) + " [default]") psf = self.config.modelPsf.apply(fwhm=fwhmPixels)