Skip to content

Commit

Permalink
Merge pull request #793 from lsst/tickets/DM-4037
Browse files Browse the repository at this point in the history
DM-4037: Require non-empty doc string for config parameters
  • Loading branch information
enourbakhsh committed Jun 9, 2023
2 parents cd7e190 + ffd9b03 commit 2df3ba6
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions python/lsst/pipe/tasks/snapCombine.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,6 @@ class SnapCombineConfig(pexConfig.Config):
doc="List of float metadata keys to average when combining snaps, e.g. float positions and dates; "
"non-float data must be handled by overriding the fixMetadata method",
optional=True,

)
sumKeys = pexConfig.ListField(
dtype=str,
Expand All @@ -113,12 +112,16 @@ class SnapCombineConfig(pexConfig.Config):
optional=True,
)

repair = pexConfig.ConfigurableField(target=RepairTask, doc="")
repair = pexConfig.ConfigurableField(target=RepairTask, doc="RepairTask configuration")
# Target `SnapPsfMatchTask` removed in DM-38846
# diffim = pexConfig.ConfigurableField(target=SnapPsfMatchTask, doc="")
detection = pexConfig.ConfigurableField(target=SourceDetectionTask, doc="")
initialPsf = pexConfig.ConfigField(dtype=InitialPsfConfig, doc="")
measurement = pexConfig.ConfigurableField(target=SingleFrameMeasurementTask, doc="")
detection = pexConfig.ConfigurableField(
target=SourceDetectionTask, doc="SourceDetectionTask configuration"
)
initialPsf = pexConfig.ConfigField(dtype=InitialPsfConfig, doc="InitialPsfConfig configuration")
measurement = pexConfig.ConfigurableField(
target=SingleFrameMeasurementTask, doc="SingleFrameMeasurementTask configuration"
)

def setDefaults(self):
self.detection.thresholdPolarity = "both"
Expand Down

0 comments on commit 2df3ba6

Please sign in to comment.