Merge pull request #2692 from cta-observatory/fix_tel_grouping

Fix telescope grouping for uint telescope id
cta-observatory · Feb 6, 2025 · f6953ee · f6953ee
2 parents 17062d4 + 03b639d
commit f6953ee
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/docs/changes/2692.bugfix.rst b/docs/changes/2692.bugfix.rst
@@ -0,0 +1,2 @@
+Fix the function grouping telescope ids into ranges for
+the case of unsigned integer telescope ids.
diff --git a/src/ctapipe/instrument/subarray.py b/src/ctapipe/instrument/subarray.py
@@ -37,7 +37,7 @@ def _group_consecutives(sequence):
     from https://codereview.stackexchange.com/questions/214820/codewars-range-extraction
     """
     sequence = sorted(sequence)
-    for _, g in groupby(enumerate(sequence), lambda i_x: i_x[0] - i_x[1]):
+    for _, g in groupby(enumerate(sequence), lambda i_x: int(i_x[0]) - int(i_x[1])):
         r = [x for _, x in g]
         if len(r) > 2:
             yield f"{r[0]}-{r[-1]}"
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Fix the function grouping telescope ids into ranges for
		the case of unsigned integer telescope ids.