Skip to content

Commit

Permalink
Merge pull request #857 from lsst/tickets/DM-41503
Browse files Browse the repository at this point in the history
DM-41503: Allow BestSeeingSelectVisitsTask to select all visits below some fwhm cut
  • Loading branch information
erykoff committed Nov 2, 2023
2 parents ea66e45 + 19708fb commit e012fd4
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions python/lsst/pipe/tasks/selectImages.py
Original file line number Diff line number Diff line change
Expand Up @@ -407,9 +407,9 @@ class BestSeeingSelectVisitsConfig(pipeBase.PipelineTaskConfig,
pipelineConnections=BestSeeingSelectVisitsConnections):
nVisitsMax = pexConfig.RangeField(
dtype=int,
doc="Maximum number of visits to select",
doc="Maximum number of visits to select; use -1 to select all.",
default=12,
min=0
min=-1,
)
maxPsfFwhm = pexConfig.Field(
dtype=float,
Expand Down Expand Up @@ -520,8 +520,11 @@ def run(self, visitSummaries, skyMap, dataId):
visits.append(visit)

sortedVisits = [ind for (_, ind) in sorted(zip(fwhmSizes, visits))]
output = sortedVisits[:self.config.nVisitsMax]
self.log.info("%d images selected with FWHM range of %d--%d arcseconds",
if self.config.nVisitsMax < 0:
output = sortedVisits
else:
output = sortedVisits[:self.config.nVisitsMax]
self.log.info("%d images selected with FWHM range of %f--%f arcseconds",
len(output), fwhmSizes[visits.index(output[0])], fwhmSizes[visits.index(output[-1])])

# In order to store as a StructuredDataDict, convert list to dict
Expand Down

0 comments on commit e012fd4

Please sign in to comment.