ess.dream: mantle DETECTOR_BANK_SIZES has counter and strip swapped
Maintainer thường phản hồi trong vòng 2 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 88/100
Hướng nghiên cứu
Bắt đầu trong ess.dream.workflows và kiểm tra DETECTOR_BANK_SIZES, so sánh thứ tự các chiều của nó với DREAM detector ICD và tệp hình học được nêu trong issue. Hoán đổi các mục counter và strip cuối cùng, sau đó xác minh rằng các id detector được gấp lại vẫn giữ các strip lân cận dọc theo z và mối quan hệ giữa các counter được mô tả trong issue.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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).
- Ngôn ngữ chính
- Python
- Star
- 1
- Fork
- 3
- Merge trung bình
- 3 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 11
Chuẩn bị môi trường
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của scipp/ess
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 2 ngày
-
essnmx good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 95/100
Maintainer thường phản hồi trong vòng 2 ngày
-
esssans
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 2 ngày
-
essdiffraction
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 2 ngày
-
[essimaging] `maximum_resolution_achievable` should not hardcode "time" as the "third" dimensionĐang mởessimaging
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 64/100
Maintainer thường phản hồi trong vòng 2 ngày
Issue tương tự
-
Claiming namespace `apoint`Đang mởnamespace operations
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 82/100
EclipseFdn/open-vsx.org#13573 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
collective/icalendar#1854 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
rancher/rancher-ai-agent#412 ·
Maintainer thường phản hồi trong vòng 6 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
TUDelftGeodesy/DePSI#134 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
HenriquesLab/rxiv-maker#335 ·