[Emc-developers] LinxCNC Python Module

[Phillip Carter]

Is the documentation correct for the LinuxCNC Python Module in regards
to kinematics_type?

In the documentation for the LinuxCNC Python Interface there is
reference to:
kinematics_type
(returns integer) - identity=1, serial=2, parallel=3, custom=4 .


in the kins manpage there is reference to:
kinstype=1 for KINEMATICS_IDENTITY (default if kinstype= omitted)
kinstype=[b|B] for KINEMATICS_BOTH
kinstype=[f|F] for KINEMATICS_FORWARD_ONLY
kinstype=[i|I] for KINEMATICS_INVERSE_ONLY


in emcmodule.cc there is reference to:
ENUM(KINEMATICS_IDENTITY);
ENUM(KINEMATICS_FORWARD_ONLY);
ENUM(KINEMATICS_INVERSE_ONLY);
ENUM(KINEMATICS_BOTH);

Cheers, Phill.

[andy pugh]

Is the documentation correct for the LinuxCNC Python Module in regards to
kinematics_type?
In the documentation for the LinuxCNC Python Interface there is reference
kinematics_type
(returns integer) - identity=1, serial=2, parallel=3, custom=4 .

This looks like a holdover from:
https://github.com/LinuxCNC/linuxcnc/blob/42ff261dfb854f7bdfbd3c29565323ee1944462e/src/emc/nml_intf/emc_nml.hh#L1060

I suspect that that entry dates back to before we had kinematics modules
and that the docs are wrong.

[Jeff Epler]

Post by Phillip Carter
Is the documentation correct for the LinuxCNC Python Module in regards
to kinematics_type?
In the documentation for the LinuxCNC Python Interface there is
kinematics_type
(returns integer) - identity=1, serial=2, parallel=3, custom=4 .

Yeah I agree that is not what the module does. Use the

Post by Phillip Carter
ENUM(KINEMATICS_IDENTITY);
ENUM(KINEMATICS_FORWARD_ONLY);
ENUM(KINEMATICS_INVERSE_ONLY);
ENUM(KINEMATICS_BOTH);

doc patches welcome.

[Jeff Epler]

Please keep replies on the list. Whole message quoted so that the
context is not lost.

Post by Phillip Carter
Is the documentation correct for the LinuxCNC Python Module in
regards to kinematics_type?
In the documentation for the LinuxCNC Python Interface there is
kinematics_type
(returns integer) - identity=1, serial=2, parallel=3, custom=4 .
Yeah I agree that is not what the module does.  Use the pseudo-enumerated
ENUM(KINEMATICS_IDENTITY);
ENUM(KINEMATICS_FORWARD_ONLY);
ENUM(KINEMATICS_INVERSE_ONLY);
ENUM(KINEMATICS_BOTH);
In commit [1]681cf92 I created a variable (trivkinstype) that I now think is
unnecessary.
Should I submit a patch that removes this variable?

Yes, I suspect it is more correct to get this value from the stat buffer
instead of from an inifile item. For one thing, the code that is
parsing the [KINS]KINEMATICS value will only work when the kinematics is
trivkins, not when it's something else like genserkins, right?

Jeff

Post by Phillip Carter
doc patches welcome.
I will look into this.