Skip to content

Some more Python Interface doc cleanup#462

Merged
c-morley merged 4 commits intoLinuxCNC:masterfrom
KurtJacobson:pyinterface-doc-cleanup
Jul 31, 2018
Merged

Some more Python Interface doc cleanup#462
c-morley merged 4 commits intoLinuxCNC:masterfrom
KurtJacobson:pyinterface-doc-cleanup

Conversation

@KurtJacobson
Copy link
Contributor

See commit messages.

In theory a joint could have different units from those set in [TRAJ] <joint_type>_UNITS if [JOINT_n] UNITS was set to something other than TRAJ units. The actual likely hood of this every occurring is very low, and may not even be possible currently. (It appears that [JOINT_n] UNITS is ignored, at least for trivkins.)
@c-morley
Copy link
Collaborator

The description says units per mm or degrees - but isn't it machine units (engineering units) ?


*max_position_limit*:: '(returns float)' -
maximum limit (soft limit) for axis motion, in machine units.configuration
maximum limit (soft limit) for axis motion, in machine units. configuration
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this say "for joint motion"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @SebKuzminsky, I did not notice that. I'm sure there are a lot more JA introduced errors hiding in plain sight! I will correct and add to this PR.


*min_position_limit*:: '(returns float)' -
minimum limit (soft limit) for axis motion, in machine units.configuration
minimum limit (soft limit) for axis motion, in machine units. configuration
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"for joint motion"?

@KurtJacobson
Copy link
Contributor Author

@c-morley The joint units will be the same as machine units for a trivkins machines, and probably many others, but it is not guaranteed to be the same as machine units. I don't fully understand it, but according to @rene-dev on IRC "non-triv machines can have different unis, like a robot. all joints are angular, but xyz abc is mm/deg"

@c-morley
Copy link
Collaborator

@KurtJacobson: I agree that I am not up on non-triv at all and could be completely wrong but,
say for instance your machine is built completely in inches - It doesn't seem right to say the joints will be in mm/degree. I would think it's really joint units/degree, joint units being what ever is set in [JOINTS]. If it is actually mm then that would be the first time that I know of that linuxcnc used a specific unit internally.
But as I said this I am talking above my pay grade :) so could be absolutely missing something.

@KurtJacobson
Copy link
Contributor Author

KurtJacobson commented Jul 26, 2018

@c-morley Now I understand your concern, I did not read your first comment properly! LinuxCNC does indeed use millimeter internally, and all other linear units are converted to millimeter. So, strange as it seems, the value of [JOINT]UNITS is the number of machine units that fit into one millimeter. For an inch machine the units value would be 0.03937in/mm, for a metric machine it would be 1mm/mm. For rotary axes the internal units is deg, so if you configure an axis in grad the units value would be 1.1111grad/deg etc. It took me quite some time to figure out was was going on, and that is what prompted me to try and clarify it in the docs.

The vast majority of the references to linuxcnc enums do not put them in single quotes, so lets be consistent.
@c-morley
Copy link
Collaborator

@KurtJacobson just revisiting your comments.
I wonder if we should add your description of how to calculate joint units - I find it non-intuitive.?

@KurtJacobson
Copy link
Contributor Author

@c-morley it probably would be good to include a description of how the joint units are calculated, but I doubt anybody even used them, so it may not be worth the effort.

@c-morley c-morley merged commit 778f2d7 into LinuxCNC:master Jul 31, 2018
@KurtJacobson
Copy link
Contributor Author

Thank you @c-morley!

@KurtJacobson KurtJacobson deleted the pyinterface-doc-cleanup branch July 31, 2018 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants