From dda51078109b4dbace05a27b7035a6dd9b90ad7e Mon Sep 17 00:00:00 2001
From: Stewart Allen <stewart.allen@gmail.com>
Date: Sat, 3 Jan 2026 16:28:18 -0500
Subject: [PATCH] drop redundant or unused index operations

---
 src/kiri/mode/cam/op-index.js  | 20 ++++++++++++++++++--
 src/kiri/mode/cam/op-rough.js  | 19 ++++++++++++++-----
 src/kiri/mode/cam/op-shadow.js |  3 +--
 src/kiri/mode/cam/slice.js     | 23 +++++++++++++++++++++--
 4 files changed, 54 insertions(+), 11 deletions(-)

diff --git a/src/kiri/mode/cam/op-index.js b/src/kiri/mode/cam/op-index.js
index 21ad46a4..46e04025 100644
--- a/src/kiri/mode/cam/op-index.js
+++ b/src/kiri/mode/cam/op-index.js
@@ -13,14 +13,30 @@ export class OpIndex extends CamOp {
 
     async slice() {
         let { op, state } = this;
+
         if (!state.isIndexed) {
             throw 'index op requires indexed stock';
         }
-        let { widget, updateSlicer, computeShadows, setAxisIndex } = state;
-        this.degrees = await setAxisIndex(op.degrees, op.absolute);
+
+        let { lastAxisIndex, widget, updateSlicer, computeShadows, setAxisIndex } = state;
+        let { degrees, absolute } = op;
+
+        if (absolute && lastAxisIndex === degrees) {
+            return console.log('skip redundant absolute index');
+        } else if (!absolute && degrees === 0) {
+            return console.log('skip redundant relative index');
+        }
+
+        this.degrees = await setAxisIndex(degrees, absolute);
+        state.lastAxisIndex = this.degrees;
+
         // force recompute of topo
         widget.topo = undefined;
+
+        // update slicer for new widget geometry
         updateSlicer();
+
+        // recompute shadow from new widget geometry
         await computeShadows();
     }
 
diff --git a/src/kiri/mode/cam/op-rough.js b/src/kiri/mode/cam/op-rough.js
index a4ce6c6f..8a1800bc 100644
--- a/src/kiri/mode/cam/op-rough.js
+++ b/src/kiri/mode/cam/op-rough.js
@@ -102,15 +102,24 @@ class OpRough extends CamOp {
             }));
         }
 
-        for (let op of this.ops_list) {
-            await op.slice(progress);
+        let len = ops_list.length;
+        let per = 1 / len;
+        let bas = 0;
+        for (let op of ops_list) {
+            await op.slice(pct => progress(bas + pct * per));
+            bas += per;
         }
     }
-s
+
     async prepare(ops, progress) {
+        let { ops_list } = this;
         let { setChangeOp } = ops;
-        for (let op of this.ops_list) {
-            await op.prepare(ops, progress);
+        let len = ops_list.length;
+        let per = 1 / len;
+        let bas = 0;
+        for (let op of ops_list) {
+            await op.prepare(ops, pct => progress(bas + pct * per));
+            bas += per;
             setChangeOp();
         }
     }
diff --git a/src/kiri/mode/cam/op-shadow.js b/src/kiri/mode/cam/op-shadow.js
index 5891e7b9..d06ce3ab 100644
--- a/src/kiri/mode/cam/op-shadow.js
+++ b/src/kiri/mode/cam/op-shadow.js
@@ -27,6 +27,7 @@ class OpShadow extends CamOp {
 
     async slice(progress) {
         let state = this.state;
+
         let { addSlices, settings, shadowAt, unsafe, widget, workarea } = state;
         let { devel } = settings.controller;
 
@@ -36,12 +37,10 @@ class OpShadow extends CamOp {
         let skipTerrain = unsafe;
         let terrain = [];
         let tzindex = [];
-        let { bottom_z, top_z } = workarea;
 
         let minZ = Math.floor(bounds.min.z);
         let maxZ = Math.floor(bounds.max.z);
         for (let z = maxZ; z >= minZ; z--) {
-            // if (z <= top_z && z >= bottom_z)
             tzindex.push(z);
         }
 
diff --git a/src/kiri/mode/cam/slice.js b/src/kiri/mode/cam/slice.js
index 8007959a..943bc300 100644
--- a/src/kiri/mode/cam/slice.js
+++ b/src/kiri/mode/cam/slice.js
@@ -55,7 +55,8 @@ export async function cam_slice(settings, widget, onupdate, ondone) {
     let proc = settings.process,
         camOps = widget.camops = [],
         sliceAll = widget.slices = [],
-        isIndexed = proc.camStockIndexed;
+        isIndexed = proc.camStockIndexed,
+        lastTime = Date.now();
 
     let axisRotation, axisIndex,
         bounds, dark, color, stock, tabs, track, tool, unsafe, units, workarea,
@@ -241,6 +242,17 @@ export async function cam_slice(settings, widget, onupdate, ondone) {
         }
     }
 
+    function updateProgress(percent, message) {
+        const mark = Date.now();
+        if (false) console.log({
+            time: ((mark - lastTime)/1000).toFixed(2),
+            pct: ((percent ?? 0) * 100).toFixed(2),
+            msg: message
+        });
+        onupdate(percent, message);
+        lastTime = mark;
+    }
+
     function newSlicer(opts) {
         return new cam_slicer(widget, opts);
     }
@@ -250,9 +262,11 @@ export async function cam_slice(settings, widget, onupdate, ondone) {
     }
 
     async function computeShadows() {
+        console.time('computeShadows');
         await new OPS.shadow(state, { type: "shadow", silent: true }).slice(progress => {
             // console.log('reshadowing', progress.round(3));
         });
+        console.timeEnd('computeShadows');
     }
 
     function setToolDiam(toolDiam) {
@@ -379,6 +393,11 @@ export async function cam_slice(settings, widget, onupdate, ondone) {
         }
     }
 
+    // drop last op if it's type index
+    if (opList.peek().op.op.type === 'index') {
+        opList.pop();
+    }
+
     // call slice() function on all ops in order
     await setAxisIndex();
     updateSlicer();
@@ -416,7 +435,7 @@ export async function cam_slice(settings, widget, onupdate, ondone) {
         });
         let operr;
         await op.slice((progress, message) => {
-            onupdate((opSum + (progress * weight)) / opTot, message || op.type());
+            updateProgress((opSum + (progress * weight)) / opTot, message || op.type());
         }).catch(e => {
             operr = e;
             console.trace(e);