Clarify that HPQ controllers means "is simulated"

lsst · May 27, 2022 · 0e607ea · 0e607ea
1 parent 478b27b
commit 0e607ea
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/python/lsst/obs/lsst/translators/lsst.py b/python/lsst/obs/lsst/translators/lsst.py
@@ -857,5 +857,13 @@ def to_has_simulated_content(self):
             if "SIMULATE" in k and v:
                 return True
 
+        # If the controller is H, P, or Q then the data are simulated.
+        ctrlr_key = "CONTRLLR"
+        if self.is_key_ok(ctrlr_key):
+            controller = self._header[ctrlr_key]
+            self._used_these_cards(ctrlr_key)
+            if controller in "HPQ":
+                return True
+
         # No simulation flags set.
         return False
diff --git a/tests/test_translator.py b/tests/test_translator.py
@@ -172,7 +172,7 @@ def test_phoSimLsstCam_translator(self):
                            exposure_time=15.0*u.s,
                            group_counter_end=32,
                            group_counter_start=32,
-                           has_simulated_content=False,  # Missing SIMULATE header.
+                           has_simulated_content=True,
                            object="UNKNOWN",
                            observation_counter=32,
                            observation_id="MC_H_20000217_000032",
@@ -269,7 +269,7 @@ def test_phoSimComCam_translator(self):
                            exposure_time=1.0*u.s,
                            group_counter_end=6001,
                            group_counter_start=6001,
-                           has_simulated_content=False,  # Should be True but the SIMULATE header is missing.
+                           has_simulated_content=True,
                            object="UNKNOWN",
                            observation_counter=6001,
                            observation_id="CC_H_20000217_006001",