step_size 来自关节轴合并的轴错误 #812

再生开发

https://github.com/LinuxCNC/linuxcnc/blob/master/src/emc/usr_intf/axis/scripts/axis.py#L3599-L3614

在联合轴合并中，
a = "XYZABCUVW"[a]
添加了该行。
python3 正确地抱怨第 3613 行
TypeError: '<' not supported between instances of 'str' and 'int'
，所以在它检查 X、Y 或 Z 轴之前。
正确的解决方法是将 if 更改为，
a in ['X','Y','Z']
但鉴于它在过去 5 年中从未正常工作（python2 总是返回 false），我无法弄清楚 step_size 实际用于什么，我想知道是否仍然需要它。

c-莫利

看起来 step_size 用于滑块的最小尺寸？但前提是缺少 INI：
DISPLAY”、”MIN_LINEAR_VELOCITY”
或：
“DISPLAY”、”MIN_VELOCITY”

安迪普

有什么理由不解决这个问题吗？
提议：

        if f != 0:
           step_size_tmp = min(step_size, 1. / f)
           if a in 'XYZ': step_size = astep_size = step_size_tmp
           elseif a in 'ABC': astep_size = step_size_tmp
           else: step_size = step_size_tmp

再生开发

我提出了一个修复方案，但由于自 JA 合并（即使使用 python2）以来它从未执行过，而且我无法弄清楚它的作用，所以我不确定是否仍然需要它。

安迪普

由b3bbb95 寻址

rene-dev 添加影响轴影响主人影响2.8 标签 2020 年 4 月 30 日

andypugh 添加了引用此问题的提交 2020 年 6 月 23 日

axis.py: Fix a hangover from the JA conversion #812

9eec6d0

andypugh已完成关闭 2020 年 6 月 23 日

step_size 来自关节轴合并的轴错误#812

注释