Do not invert for Pilatus 4 (#663)

* Do not invert for Pilatus 4 * Rename newsfragments/XXX.bugfix to newsfragments/663.bugfix --------- Co-authored-by: DiamondLightSource-build-server <[email protected]>
cctbx · Nov 2, 2023 · 05b7806 · 05b7806
1 parent 4ad0ce5
commit 05b7806
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/newsfragments/663.bugfix b/newsfragments/663.bugfix
@@ -0,0 +1,2 @@
+Pilatus 4: do not invert module size (is written correctly in master file)
+
diff --git a/src/dxtbx/format/FormatNXmxEigerFilewriter.py b/src/dxtbx/format/FormatNXmxEigerFilewriter.py
@@ -52,8 +52,10 @@ def _get_nxmx(self, fh: h5py.File):
         # data_size is reversed - we should probably be more specific in when
         # we do this, i.e. check data_size is in a list of known reversed
         # values
+        known_safe = [(1082,1035),]
         for module in nxdetector.modules:
-            module.data_size = module.data_size[::-1]
+            if not tuple(module.data_size) in known_safe:
+                module.data_size = module.data_size[::-1]
         return nxmx_obj
 
     def get_raw_data(self, index):