Merge pull request #2431 from LukasBeiske/check_coord_frame

ctapipe/tools/train_disp_reconstructor.py

@@ -1,6 +1,7 @@
 import astropy.units as u
 import numpy as np
 
+from ctapipe.containers import CoordinateFrameType
 from ctapipe.core import Tool
 from ctapipe.core.traits import Bool, Int, IntTelescopeParameter, Path
 from ctapipe.exceptions import TooFewEvents
@@ -127,6 +128,13 @@ def _read_table(self, telescope_type):
                 f"No events after quality query for telescope type {telescope_type}"
             )
 
+        if not np.all(
+            table["subarray_pointing_frame"] == CoordinateFrameType.ALTAZ.value
+        ):
+            raise ValueError(
+                "Pointing information for training data has to be provided in horizontal coordinates"
+            )
+
         table = self.models.feature_generator(table, subarray=self.loader.subarray)
 
         table[self.models.target] = self._get_true_disp(table)

docs/changes/2431.bugfix.rst

@@ -0,0 +1,2 @@
+Check that the array pointing is given in horizontal coordinates
+before training a ``DispReconstructor``.