SANS: Handle leading/trailing whitespace in sample position strings

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#120 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

@dehoni đang làm issue này rồi.

Từ ngày 11/8/2026.

  • #121 của @dehoni — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
68/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
python
Lĩnh vực
tooling

Hướng nghiên cứu

Truy vết điểm vào để xác thực và tra cứu vị trí mẫu SANS; issue không nêu tệp hoặc test nào. Tái hiện hành vi với các vị trí như " 1CB" và "1CB ", sau đó xác minh rằng khoảng trắng xung quanh không còn khiến một vị trí được yêu cầu bị bỏ qua và rằng phạm vi kiểm thử xác thực hiện có đều đạt.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Users occasionally edit measurement scripts manually and enter sample position strings with leading or trailing whitespace, for example: " 1CB" or
"1CB ". This is easy to do inadvertently and is often intended for readability rather than representing a different sample position.

Currently, when a position string contains extra whitespace, it is not recognised as a valid sample position. The issue is only detected when the script attempts to move the sample changer. At that point, the command is silently ignored because the requested position is not found in the list of valid positions. The run continues, resulting in fewer samples being measured than expected.

This behaviour is particularly problematic for unattended, overnight measurements, where users may only discover later that part of the sample sequence was skipped.

Ngôn ngữ chính
Python
Star
4
Fork
8
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

  • Không có Dockerfile hay tệp Docker Compose
  • Có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của ISISNeutronMuon/InstrumentScripts

Tất cả issue của ISISNeutronMuon/InstrumentScripts

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.