Skip to content

Commit

Permalink
Merge pull request #954 from lsst/tickets/DM-45131
Browse files Browse the repository at this point in the history
DM-45131: Remove lsst.log mentions that aren't needed
  • Loading branch information
timj authored Jul 8, 2024
2 parents de2a100 + 09d37cb commit 67c8e60
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion python/lsst/pipe/tasks/insertFakes.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ def _add_fake_sources(exposure, objects, calibFluxRadius=12.0, logger=None):
exposure+catalog. This is used to produce the correct instrumental fluxes
within the radius. The value should match that of the field defined in
slot_CalibFlux_instFlux.
logger : `lsst.log.log.log.Log` or `logging.Logger`, optional
logger : `logging.Logger`, optional
Logger.
"""
exposure.mask.addMaskPlane("FAKE")
Expand Down
1 change: 0 additions & 1 deletion tests/test_calibrateImage.py
Original file line number Diff line number Diff line change
Expand Up @@ -646,7 +646,6 @@ def mock_run(exposures, result=None, id_generator=None):
with (
mock.patch.object(task, "run", side_effect=mock_run),
self.assertRaises(lsst.pipe.base.AnnotatedPartialOutputsError),
lsst.log.UsePythonLogging(), # so that assertLogs works with lsst.log
):
with self.assertLogs("lsst.calibrateImage", level="ERROR") as cm:
lsst.pipe.base.testUtils.runTestQuantum(task,
Expand Down
1 change: 0 additions & 1 deletion ups/pipe_tasks.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import lsst.sconsUtils
dependencies = dict(
required = [
"afw",
"log",
"meas_base",
"pex_config",
],
Expand Down

0 comments on commit 67c8e60

Please sign in to comment.