Hi there,

after reading the patch I have following observations:

emc2-unhome/src/emc/motion/command.c:
* you do check for homing (correctly) when unhoming a single joint, but no 
check is done in the case -1 (unhome all joints) is issued
* it would be nice if you could stick to the same formatting as the rest of 
the file ({} after the if (), } else { etc)

emc2-unhome/src/emc/motion/motion.h:
* you missed a comment there: + int unhome;

The rest seems ok (hope it applies).

Regards,
Alex



----- Original Message ----- 
From: [EMAIL PROTECTED]
To: emc-developers@lists.sourceforge.net
Sent: Friday, February 15, 2008 4:45 PM
Subject: [Emc-developers] 'Un-home' capability


Dear fellow developers,
The attached patch file adds the ability to un-home an axis (clear its homed 
flag), namely through the new [AXIS_n]VOLATILE_HOME ini file option, the 
EMC_AXIS_UNHOME NML message, and the emcAxisUnhome(int axis) call. See 
http://www.linuxcnc.org/irc/irc.freenode.net:6667/emc/2008-02-14.txt and 
http://www.linuxcnc.org/irc/irc.freenode.net:6667/emc/2008-02-15.txt on the 
discussion of "unhom*".

After applying and (re-)compiling, setting VOLATILE_HOME to 1 for any 
particular axis will cause it to lose its homed flag upon ESTOP and OFF.

The syntax for the milltask API call is almost identical to its homing 
counterpart. Setting the "axis" variable to n unhomes the n-th joint; 
setting it to -1, unhomes all joints with the permitting ini entry. Ditto 
for the NML message.

jepler suggested that the ini file permit three possible values for 
VOLATILE_HOME: "never," "amplifier-off," and "estop." SWPadnos emphasized 
the ability to cause unhoming from a HAL module. It should be relatively 
easy to move the emcAxisUnhome calls around inside emcTaskSetState to 
satisfy the former. As for the latter, an "unhome" pin could be added to 
ioControl.cc (or halusr?) which, when triggered, cause it to send the 
EMC_AXIS_UNHOME packet. I implemented neither of the two in my patch because 
my original purpose was simply to provide the function calls upon which this 
feature could be further developed.


Enjoy,
Bryant




-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/



_______________________________________________
Emc-developers mailing list
Emc-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/emc-developers 


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Emc-developers mailing list
Emc-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/emc-developers

Reply via email to