ess.dream: mantle DETECTOR_BANK_SIZES has counter and strip swapped
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 88/100
Línea de trabajo
Comienza en ess.dream.workflows e inspecciona DETECTOR_BANK_SIZES, comparando el orden de sus dimensiones con el DREAM detector ICD y el archivo de geometría mencionado en la issue. Intercambia las entradas finales de counter y strip, y luego verifica que los ids de detector plegados conserven los strips vecinos a lo largo de z y la relación entre counters descrita en la issue.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
DETECTOR_BANK_SIZES in ess.dream.workflows folds the mantle as
{"wire": 32, "module": 5, "segment": 6, "strip": 256, "counter": 2}
but the DREAM detector ICD (ESS-5462547, section 4.4.3) defines pixel = 256·(60·wire + 12·MU + 2·cassette + counter) + strip + 1, i.e. C-order (wire, module, segment, counter, strip). The geometry file (geometry-dream-no-shape-2026-06-09.nxs) agrees with the ICD: consecutive ids are neighbouring strips along z, and id + 256 is the other counter.
With the current order, folded strip jumps from z = +1078 mm back to -1091 mm at index 128, and counter is strip parity (adjacent strips 13 mm apart in z, same azimuth). Positions are folded with the same order, so anything that only uses coordinates is unaffected; anything selecting or labelling by strip/counter is not.
Fix: swap the last two entries.
Found while fixing the live-data logical views in scipp/esslivedata (they no longer use this dict).
- Lenguaje dominante
- Python
- Estrellas
- 1
- Forks
- 3
- Merge medio
- 3 d 19 h
- PR fusionados (30 d)
- 11
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de scipp/ess
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 2 días
-
essnmx good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
scipp/ess#760 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
esssans
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 2 días
-
essdiffraction
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 2 días
-
[essimaging] `maximum_resolution_achievable` should not hardcode "time" as the "third" dimensionAbiertoessimaging
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Los mantenedores suelen responder en 2 días
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
PedestrianDynamics/pyFDS-Evac#343 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
theskumar/python-dotenv#708 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 2 días
-
Docs Timedelta
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
pandas-dev/pandas#69919 ·
Los mantenedores suelen responder en 1 día
-
API documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
zephyrproject-rtos/west#1009 · 2 comentarios ·
Los mantenedores suelen responder en 3 días