SANS: Handle leading/trailing whitespace in sample position strings

未关闭 适合新手
#120 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

@dehoni 已经在做这个了。

开始于 2026年8月11日。

  • #121 来自 @dehoni —— 未关闭

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
68/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
冷清
技术栈
python
领域
tooling

调研方向

跟踪 SANS 样本位置验证和查找的入口点;该 issue 没有指定文件或测试。使用 " 1CB" 和 "1CB " 等位置重现该行为,然后验证周围的空白不再导致请求的位置被跳过,并确认现有的验证覆盖率测试通过。

由索引模型根据 Issue 内容生成。

描述

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.

主要语言
Python
星标
4
派生
8
PR 合并指标
30 天内没有已合并 PR

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 有 Pull Request 模板
  • 没有贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

ISISNeutronMuon/InstrumentScripts 的其他 Issue

查看 ISISNeutronMuon/InstrumentScripts 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。