DM-47990: Create a new action group dedicated to HealSparse maps and relocate LoadHealSparseMap() from tools to actions

python/lsst/analysis/tools/actions/healSparseMap/__init__.py

@@ -0,0 +1 @@
+from .healSparseMapActions import *

python/lsst/analysis/tools/actions/healSparseMap/healSparseMapActions.py

@@ -0,0 +1,42 @@
+# This file is part of analysis_tools.
+#
+# Developed for the LSST Data Management System.
+# This product includes software developed by the LSST Project
+# (https://www.lsst.org).
+# See the COPYRIGHT file at the top-level directory of this distribution
+# for details of code ownership.
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <https://www.gnu.org/licenses/>.
+from __future__ import annotations
+
+__all__ = ("LoadHealSparseMap",)
+
+from typing import cast
+
+from healsparse.healSparseMap import HealSparseMap
+from lsst.pex.config import Field
+
+from ...interfaces import HealSparseMapAction, KeyedData, KeyedDataSchema
+
+
+class LoadHealSparseMap(HealSparseMapAction):
+    """Load and return a HealSparseMap from KeyedData."""
+
+    mapKey = Field[str](doc="Key of map which should be loaded.")
+
+    def getInputSchema(self) -> KeyedDataSchema:
+        return ((self.mapKey, HealSparseMap),)
+
+    def __call__(self, data: KeyedData, **kwargs) -> HealSparseMap:
+        return cast(HealSparseMap, data[self.mapKey])

python/lsst/analysis/tools/atools/healSparsePropertyMap.py

@@ -25,25 +25,11 @@
     "SurveyWidePropertyMapTool",
 )
 
-from typing import cast
-
-from healsparse.healSparseMap import HealSparseMap
 from lsst.pex.config import Field
 
+from ..actions.healSparseMap.healSparseMapActions import LoadHealSparseMap
 from ..actions.plot.propertyMapPlot import PerTractPropertyMapPlot, SurveyWidePropertyMapPlot
-from ..interfaces import AnalysisAction, AnalysisTool, KeyedData, KeyedDataSchema
-
-
-class LoadHealSparseMap(AnalysisAction):
-    """Load and return a HealSparseMap from KeyedData."""
-
-    mapKey = Field[str](doc="Key of map which should be loaded.")
-
-    def getInputSchema(self) -> KeyedDataSchema:
-        return ((self.mapKey, HealSparseMap),)
-
-    def __call__(self, data: KeyedData, **kwargs) -> HealSparseMap:
-        return cast(HealSparseMap, data[self.mapKey])
+from ..interfaces import AnalysisTool
 
 
 class PerTractPropertyMapTool(AnalysisTool):

python/lsst/analysis/tools/interfaces/_actions.py

@@ -23,6 +23,7 @@
 
 __all__ = (
     "AnalysisAction",
+    "HealSparseMapAction",
     "KeyedDataAction",
     "VectorAction",
     "ScalarAction",
@@ -43,6 +44,7 @@
 from typing import TYPE_CHECKING, Iterable
 
 import lsst.pex.config as pexConfig
+from healsparse.healSparseMap import HealSparseMap
 from lsst.pex.config.configurableActions import ConfigurableAction, ConfigurableActionField
 
 from ..contexts import ContextApplier
@@ -139,6 +141,16 @@ def addInputSchema(self, inputSchema: KeyedDataSchema) -> None:
         )
 
 
+class HealSparseMapAction(AnalysisAction):
+    """A `HealSparseMapAction` is an `AnalysisAction` that returns a
+    `HealSparseMap` when called.
+    """
+
+    @abstractmethod
+    def __call__(self, data: KeyedData, **kwargs) -> HealSparseMap:
+        raise NotImplementedError("This is not implemented on the base class")
+
+
 class KeyedDataAction(AnalysisAction):
     """A `KeyedDataAction` is an `AnalysisAction` that returns `KeyedData` when
     called.