AI_Assistant/grid-apps-cmms
Watch
1
0
Fork
You've already forked grid-apps-cmms
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix order of operations bug with tabs and inner only

Browse source
This commit is contained in:
Stewart Allen 2025-12-20 13:53:40 -05:00
commit d01f4a1300
1 changed files with 8 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/kiri/mode/cam/op-area.js
View file

@ -181,6 +181,12 @@ class OpArea extends CamOp {
// terminate z descent when no further output possible
break outer;
}
// support legacy outline features
if (op.omitouter) {
outs = omitOuter(outs);
} else if (op.omitinner) {
outs = omitInner(outs);
}
// cut tabs when present
if (tabs.length) outs = cutTabs(tabs, outs);
// for roughing backward compatability
@ -189,12 +195,6 @@ class OpArea extends CamOp {
for (let p of out.points)
p.z += op.leave_z;
}
// support legacy outline features
if (op.omitouter) {
outs = omitOuter(outs);
} else if (op.omitinner) {
outs = omitInner(outs);
}
// add tabs to travel boundaries
if (tabs.length) {
let tab_shadows = tabs.filter(t => t.top >= z).map(t => t.poly);
@ -259,14 +259,14 @@ class OpArea extends CamOp {
}
// add dogbones when specified
if (op.dogbones) outs.forEach(out => out.addDogbones(toolDiam / 5, op.revbones));
// cut tabs when present
if (tabs.length) outs = cutTabs(tabs, outs);
// support legacy outline features
if (op.omitouter) {
outs = omitOuter(outs);
} else if (op.omitinner) {
outs = omitInner(outs);
}
// cut tabs when present
if (tabs.length) outs = cutTabs(tabs, outs);
slice.camLines = outs;
POLY.setWinding(outs, direction === 'climb');
// store travel boundary that triggers up and over moves