Skip to content

Commit 81e1046

Browse files
authored
Merge pull request #415 from lsst/tickets/DM-51008
DM-51008: Ensure that fully vignetted pixels and variances are set to 0
2 parents cef618a + d764ca1 commit 81e1046

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

python/lsst/ip/isr/isrTaskLSST.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2239,6 +2239,7 @@ def run(self, ccdExposure, *, dnlLUT=None, bias=None, deferredChargeCalib=None,
22392239

22402240
noData = (ccdExposure.mask.array & ccdExposure.mask.getPlaneBitMask("NO_DATA")) > 0
22412241
ccdExposure.image.array[noData] = 0.0
2242+
ccdExposure.variance.array[noData] = 0.0
22422243

22432244
ccdExposure.metadata["LSST ISR FLAT APPLIED"] = True
22442245
ccdExposure.metadata["LSST ISR FLAT SOURCE"] = flat.metadata.get("FLATSRC", "UNKNOWN")

tests/test_isrTaskLSST.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1174,6 +1174,9 @@ def test_isrFlatVignette(self):
11741174
noDataExp = (result.exposure.mask.array & result.exposure.mask.getPlaneBitMask("NO_DATA")) > 0
11751175
np.testing.assert_array_equal(noDataExp, noDataFlat)
11761176
np.testing.assert_array_equal(result.exposure.image.array[noDataExp], 0.0)
1177+
np.testing.assert_array_equal(result.exposure.variance.array[noDataExp], 0.0)
1178+
self.assertFalse(np.any(~np.isfinite(result.exposure.image.array)))
1179+
self.assertFalse(np.any(~np.isfinite(result.exposure.variance.array)))
11771180

11781181
def test_isrFloodedSaturatedE2V(self):
11791182
"""Test ISR when the amps are completely saturated.

0 commit comments

Comments
 (0)