Merge pull request #38 from scipp/fix-chopper-check

fix: don't assume log has value field
scipp · Oct 28, 2024 · 7cfa07c · 7cfa07c
2 parents d642376 + 63a94c7
commit 7cfa07c
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/chexus/validators.py b/src/chexus/validators.py
@@ -244,10 +244,15 @@ def validate(self, node: Dataset | Group) -> Violation | None:
         import scipp as sc
 
         rotation_speed = node.children.get("rotation_speed")
-        if "NXlog" == rotation_speed.attrs.get("NX_class"):
-            unit = rotation_speed.children.get("value").attrs.get("units")
-        else:
+        if (
+            "NXlog" == rotation_speed.attrs.get("NX_class")
+            and "value" in rotation_speed.children
+        ):
+            unit = rotation_speed.children["value"].attrs.get("units")
+        elif "NXlog" != rotation_speed.attrs.get("NX_class"):
             unit = rotation_speed.attrs.get("units")
+        else:
+            return
         try:
             sc.scalar(1, unit=unit).to(unit="Hz")
         except sc.UnitError: