Fix potential crash in indexing max cell estimation if all spots are …

…ice ring resolution (dials#2699)
toastisme · Jul 12, 2024 · 153f5a7 · 153f5a7
1 parent 1848e04
commit 153f5a7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/newsfragments/2699.bugfix b/newsfragments/2699.bugfix
@@ -0,0 +1 @@
+``dials.index``: Fix potential crash in max_cell estimation when all ice ring spots
diff --git a/src/dials/algorithms/indexing/max_cell.py b/src/dials/algorithms/indexing/max_cell.py
@@ -33,6 +33,10 @@ def find_max_cell(
         logger.debug(
             "Rejecting %i reflections at ice ring resolution", ice_sel.count(True)
         )
+        if not reflections.size():
+            raise DialsIndexError(
+                "No spots left for max cell analysis after ice-ring filtering"
+            )
 
     # need bounding box in reflections to find overlaps; this is not there if
     # spots are from XDS (for example)