Skip to content

Commit

Permalink
Use 'pvi', not 'initial_pvi', in forcedPhotCcd.
Browse files Browse the repository at this point in the history
We also turn off using the finalVisitSummary input, which is important
for the PhotoCalib, since both pvi and initial_pvi have their pixel
values calibrated to nJy and their PhotoCalib reset to the identity.
This means forcedPhotCcd's instFlux columns will actually have nJy
units, but this is fine since their "DN" is already an arbitrary unit
and the LocalPhotoCalib column will reflect this by being full of ones.
  • Loading branch information
TallJimbo committed Oct 9, 2024
1 parent 308b7ae commit 80fcb17
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions python/lsst/meas/base/forcedPhotCcd.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
class ForcedPhotCcdConnections(
PipelineTaskConnections,
dimensions=("instrument", "visit", "detector", "skymap", "tract"),
defaultTemplates={"inputCoaddName": "deep", "inputName": "initial_pvi"},
defaultTemplates={"inputCoaddName": "deep", "inputName": "pvi"},
):
inputSchema = cT.InitInput(
doc="Schema for the input measurement catalogs.",
Expand Down Expand Up @@ -140,7 +140,7 @@ class ForcedPhotCcdConfig(pipeBase.PipelineTaskConfig, pipelineConnections=Force
)
useVisitSummary = lsst.pex.config.Field(
dtype=bool,
default=True,
default=False,
doc=(
"Use updated WCS, PhotoCalib, ApCorr, and PSF from visit summary? "
"This should be False if and only if the input image already has the best-available calibration "
Expand Down Expand Up @@ -465,7 +465,7 @@ class ForcedPhotCcdFromDataFrameConnections(
dimensions=("instrument", "visit", "detector", "skymap", "tract"),
defaultTemplates={
"inputCoaddName": "goodSeeing",
"inputName": "initial_pvi",
"inputName": "pvi",
},
):
refCat = cT.Input(
Expand Down

0 comments on commit 80fcb17

Please sign in to comment.