Skip to content

Commit

Permalink
Remove pex_logging dependency
Browse files Browse the repository at this point in the history
In tests/testKron.py, remove the unused placeholder for the
"meas.photometry.kron" logger, as there are no messages logged
to this logger shown up by running this test, before any
logging framework migration.
  • Loading branch information
Hsin-Fang Chiang committed Nov 22, 2016
1 parent 6033008 commit 7db96b1
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 6 deletions.
1 change: 0 additions & 1 deletion python/lsst/meas/extensions/photometryKron/kronLib.i
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ Interface to Kron magnitudes

%{
#include "lsst/base.h"
#include "lsst/pex/logging.h"
#include "lsst/afw/detection/Footprint.h"
#include "lsst/afw/image/Utils.h"
#include "lsst/afw/geom/Point.h"
Expand Down
3 changes: 0 additions & 3 deletions tests/testKron.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@
import numpy as np
import itertools
import lsst.utils.tests
import lsst.pex.logging as pexLogging
import lsst.afw.detection as afwDetection
import lsst.afw.geom as afwGeom
import lsst.afw.geom.ellipses as afwEllipses
Expand All @@ -47,10 +46,8 @@
try:
type(verbose)
except NameError:
verbose = 1
display = False
ds9Frame = 0
pexLogging.Trace_setVerbosity("meas.photometry.kron", verbose)

import lsst.afw.display.ds9 as ds9
import lsst.afw.display.utils as displayUtils
Expand Down
2 changes: 1 addition & 1 deletion ups/meas_extensions_photometryKron.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
import lsst.sconsUtils

dependencies = {
"required": ["utils", "afw", "meas_base", "pex_logging"],
"required": ["utils", "afw", "meas_base"],
"buildRequired": ["boost_test", "swig"],
}

Expand Down
1 change: 0 additions & 1 deletion ups/meas_extensions_photometryKron.table
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
setupRequired(meas_base)
setupRequired(pex_logging)
setupOptional(meas_algorithms) # used by a unit test

envPrepend(LD_LIBRARY_PATH, ${PRODUCT_DIR}/lib)
Expand Down

0 comments on commit 7db96b1

Please sign in to comment.