Skip to content

Commit

Permalink
Remove testKernelSize
Browse files Browse the repository at this point in the history
  • Loading branch information
arunkannawadi committed Jun 30, 2023
1 parent f48a6b9 commit fbffce5
Showing 1 changed file with 0 additions and 24 deletions.
24 changes: 0 additions & 24 deletions tests/test_psf.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@
from lsst.meas.base import SingleFrameMeasurementTask
from lsst.meas.extensions.piff.piffPsfDeterminer import PiffPsfDeterminerConfig, PiffPsfDeterminerTask
from lsst.meas.extensions.piff.piffPsfDeterminer import _validateGalsimInterpolant
from lsst.pex.config import FieldValidationError


def psfVal(ix, iy, x, y, sigma1, sigma2, b):
Expand Down Expand Up @@ -371,29 +370,6 @@ def testValidateGalsimInterpolant(self):
self.assertTrue(_validateGalsimInterpolant(f"galsim.{interp}"))
self.assertTrue(eval(f"galsim.{interp}"))

def testKernelSize(self): # TODO: Remove this test in DM-36311.
config = PiffPsfDeterminerConfig()

# Setting both stampSize and kernelSize should raise an error.
config.stampSize = 27
with self.assertWarns(FutureWarning):
config.kernelSize = 25
self.assertRaises(FieldValidationError, config.validate)

# even if they agree with each other
config.stampSize = 31
with self.assertWarns(FutureWarning):
config.kernelSize = 31
self.assertRaises(FieldValidationError, config.validate)

# Setting stampSize and kernelSize should be valid, because if not
# set, stampSize is set to the size of PSF candidates internally.
# This is only a temporary behavior and should go away in DM-36311.
config.stampSize = None
with self.assertWarns(FutureWarning):
config.kernelSize = None
config.validate()


class TestMemory(lsst.utils.tests.MemoryTestCase):
pass
Expand Down

0 comments on commit fbffce5

Please sign in to comment.