Skip to content

Commit

Permalink
Add requireFiniteRaDec clause to source selector
Browse files Browse the repository at this point in the history
We are now persisting source catalogs even in light of an astrometric
failure in single frame processing.  Such catalogs will have their sky
coordinate column (typically "coord_ra" and "coord_dec") values set to
NaN.  They are thus not suitable for inclusion in the fgcm calibration,
so this config default ensures they are omited from the selection of
"good" sources.
  • Loading branch information
laurenam committed Apr 15, 2022
1 parent 2e4e590 commit c002342
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions python/lsst/fgcmcal/fgcmBuildStarsBase.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,6 +186,7 @@ def setDefaults(self):
sourceSelector.doUnresolved = True
sourceSelector.doSignalToNoise = True
sourceSelector.doIsolated = True
sourceSelector.doRequireFiniteRaDec = True

sourceSelector.signalToNoise.minimum = 10.0
sourceSelector.signalToNoise.maximum = 1000.0
Expand Down
3 changes: 3 additions & 0 deletions python/lsst/fgcmcal/fgcmBuildStarsTable.py
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,9 @@ def setDefaults(self):
sourceSelector.isolated.parentName = 'parentSourceId'
sourceSelector.isolated.nChildName = 'deblend_nChild'

sourceSelector.requireFiniteRaDec.raColName = 'ra'
sourceSelector.requireFiniteRaDec.decColName = 'decl'

sourceSelector.unresolved.name = 'extendedness'


Expand Down

0 comments on commit c002342

Please sign in to comment.