LinuxCNC/linuxcnc

Tool table widget: expand for column "rotation"

hansu opened this issue · 9 comments

I wonder if there was a specific reason to display the rotational offset not as a column for every coordinate system.
Was it just a matter of space and the rare use case?
@c-morley ?

Current:
grafik

Proposal:
grafik

I agree. This makes much more sense as each WCS has it's own 'R' value.

And having a spurious rotation is an occasional cause of trouble. (I saw a recent forum post where someone said that their X and Y axes had swapped. That was the cause.

Does each system have it's own rotation? I thought the rotation was was separate and added after the user system.

Yes, each system has it's own rotation. AxisGUI displays R values correctly for each system.

What python attribute gives the rotation setting for systems not in use?
rotation_xy only gives the current rotation.

As far as I'm aware there are only python attributes for for the currently active WCS system. I think one would have to query parameters #5230 + i*20, i=[0..8]

Ahh yes ,it's coming back to me now. We read the var file directly for the other user systems. I assume there is a var in that file for the other rotations too.

@c-morley Are you currently working on this? If yes - I thought it would be nice if we could use the calculator widget to enter the offset values. What do you think?