Offset page widget: expand for column "rotation"

[hansu]

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:

Proposal:

[Sigma1912]

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

[andypugh]

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.

[c-morley]

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

[Sigma1912]

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

[hansu]

Yes, except G92. See https://linuxcnc.org/docs/devel/html/gcode/g-code.html#gcode:g10-l2

[c-morley]

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

[Sigma1912]

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]

[c-morley]

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.

[hansu]

@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?

[hansu]

Fixed by #3457