Merge pull request #2616 from cta-observatory/site_string

Convert metadata Instrument.site to free string instead of enum
cta-observatory · Sep 12, 2024 · 80914fb · 80914fb
2 parents 4b85d1f + 35ad049
commit 80914fb
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 16 deletions.
diff --git a/docs/changes/2616.feature.rst b/docs/changes/2616.feature.rst
@@ -0,0 +1,2 @@
+The ``Instrument.site`` metadata item now accepts any string,
+not just a pre-defined list of sites.
diff --git a/src/ctapipe/io/metadata.py b/src/ctapipe/io/metadata.py
@@ -176,23 +176,14 @@ def default_time(self):
 class Instrument(Configurable):
     """Instrumental Context"""
 
-    site = Enum(
-        [
-            "CTA-North",
-            "CTA-South",
-            "SDMC",
-            "HQ",
-            "MAGIC",
-            "HESS",
-            "VERITAS",
-            "FACT",
-            "Whipple",
-            "Other",
-        ],
-        "Other",
-        help="Which site of CTA (or external telescope) "
-        "this instrument is associated with",
+    site = Unicode(
+        default_value="Other",
+        help=(
+            "Which site of CTAO (or external telescope)"
+            " this instrument is associated with"
+        ),
     ).tag(config=True)
+
     class_ = Enum(
         [
             "Array",