Merge pull request #15 from lsst/tickets/DM-31062

DM-31062: Use the plugin name by default
lsst · Jul 10, 2021 · 26e77d7 · 26e77d7
2 parents 8227c59 + fde6208
commit 26e77d7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/python/lsst/meas/extensions/gaap/_gaap.py b/python/lsst/meas/extensions/gaap/_gaap.py
@@ -219,7 +219,7 @@ def _getGaapResultName(scalingFactor: float, sigma: Union[float, str], name: Opt
             return suffix
         return "_".join((name, suffix))
 
-    def getAllGaapResultNames(self, name: Optional[str] = None) -> Generator[str]:
+    def getAllGaapResultNames(self, name: Optional[str] = PLUGIN_NAME) -> Generator[str]:
         """Generate the base names for all of the GAaP fields.
 
         For example, if the plugin is configured with `scalingFactors` = [1.15]

diff --git a/tests/test_gaap.py b/tests/test_gaap.py
@@ -640,7 +640,7 @@ def testMonteCarlo(self, noise, recordId=1, sigmas=[0.7, 1.0, 1.25],
             catalog.append(record)
 
         catalog = catalog.copy(deep=True)
-        for baseName in gaapConfig.getAllGaapResultNames(name="ext_gaap_GaapFlux"):
+        for baseName in gaapConfig.getAllGaapResultNames():
             instFluxKey = schema.join(baseName, "instFlux")
             instFluxErrKey = schema.join(baseName, "instFluxErr")
             instFluxMean = catalog[instFluxKey].mean()