Merge pull request #244 from lsst/tickets/DM-39141

DM-39141: Add doRequirePrimary to star selector.
lsst · May 26, 2023 · b2c6b38 · b2c6b38
2 parents 93dcef9 + 082f0dd
commit b2c6b38
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/python/lsst/jointcal/jointcal.py b/python/lsst/jointcal/jointcal.py
@@ -505,6 +505,8 @@ def validate(self):
             logging.getLogger("lsst.jointcal").warning(msg)
 
     def setDefaults(self):
+        # Use only primary stars.
+        self.sourceSelector["science"].doRequirePrimary = True
         # Use only stars because aperture fluxes of galaxies are biased and depend on seeing.
         self.sourceSelector["science"].doUnresolved = True
         self.sourceSelector["science"].unresolved.name = "extendedness"
@@ -1604,6 +1606,8 @@ def get_sourceTable_visit_columns(inColumns, config, sourceSelector):
     if sourceSelector.config.doRequireFiniteRaDec:
         columns.append(sourceSelector.config.requireFiniteRaDec.raColName)
         columns.append(sourceSelector.config.requireFiniteRaDec.decColName)
+    if sourceSelector.config.doRequirePrimary:
+        columns.append(sourceSelector.config.requirePrimary.primaryColName)
 
     return columns, ixxColumns
 

diff --git a/tests/config/config.py b/tests/config/config.py
@@ -2,3 +2,5 @@
 
 # jointcal test metrics were made including unresolved sources.
 config.sourceSelector['science'].doUnresolved = False
+# Test catalogs do not have detect_isPrimary
+config.sourceSelector['science'].doRequirePrimary = False