operation with multiplie sketches in pocket - tool path revisits sketches

[GoogleCodeExporter]

See attached file of four rectangles all in one pocket operation.  Instead of 
completing each rectangle and then moving to the next one, the pocketing 
operation jumps between rectangles multiple times.  I noticed this change in 
behavior in version 0.17.2 of heekscnc.  When I did this same operation in an 
older version (0.15) it generates the toolpath you'd expect - visiting each 
rectangle once.

Original issue reported on code.google.com by filipmul...@q.com on 18 Mar 2011 at 3:23

Attachments:

• [multiple pocket defect.heeks](https://storage.googleapis.com/google-code-attachments/heekscnc/issue-294/comment-0/multiple pocket defect.heeks)

[GoogleCodeExporter]

witch G-code postprocessor did you used ?

Original comment by lot...@foengarage.de on 24 Mar 2011 at 6:44

[GoogleCodeExporter]

I used the Mach 3 machine.  I also get the same incorrect results with the 
Standard ISO and the Linux EMC2.

Original comment by filipmul...@q.com on 25 Mar 2011 at 1:57

[GoogleCodeExporter]

Quick fix, not extensively tested:
--- area_funcs.py.1327  2011-05-07 16:06:08.000000000 +0300
+++ area_funcs.py   2011-08-03 15:28:16.953125000 +0300
@@ -340,6 +340,16 @@
 def pocket(a, tool_radius, extra_offset, rapid_safety_space, start_depth, final_depth, stepover, stepdown, clearance_height, from_center, keep_tool_down_if_poss, use_zig_zag, zig_angle, zig_unidirectional = False):
     global tool_radius_for_pocket
     global area_for_feed_possible
+    
+    
+    if len(a.getCurves()) > 1:
+        for crv in a.getCurves():
+            ar = area.Area()
+            ar.append(crv)
+            pocket(ar, tool_radius, extra_offset, rapid_safety_space, 
start_depth, final_depth, stepover, stepdown, clearance_height, from_center, 
keep_tool_down_if_poss, use_zig_zag, zig_angle, zig_unidirectional)
+        return
+
+
     tool_radius_for_pocket = tool_radius

     if keep_tool_down_if_poss:

Original comment by attem...@gmail.com on 3 Aug 2011 at 12:29

[GoogleCodeExporter]

This looks like a good fix to me, so I have added it to the source code.
Source code is now here: https://github.com/Heeks/heekscnc

Original comment by danhe...@gmail.com on 3 Aug 2011 at 12:49

• Changed state: Fixed

[GoogleCodeExporter]

This seems to have reappeared in 0.22.0 ?
I used multiple archs defining circles, one for each scetch and selected 
multiple scetches. Result is a all scetches-first approach instead of depth 
first both with and without "keep tool down" ticked.

Original comment by kyrr...@gmail.com on 23 Sep 2012 at 3:33