From 2c35de71bf236dff81240e9ea94989d04c5ada47 Mon Sep 17 00:00:00 2001
From: Stewart Allen <stewart.allen@gmail.com>
Date: Tue, 28 May 2019 11:52:14 -0400
Subject: [PATCH] remove dump. update notes

---
 js/geo-polygon.js     | 27 ---------------------------
 js/geo-polygons.js    | 11 -----------
 js/kiri-driver-fdm.js |  1 +
 notes.md              |  7 ++-----
 4 files changed, 3 insertions(+), 43 deletions(-)

diff --git a/js/geo-polygon.js b/js/geo-polygon.js
index c32d79c6..2ff2b754 100644
--- a/js/geo-polygon.js
+++ b/js/geo-polygon.js
@@ -916,33 +916,6 @@ var gs_base_polygon = exports;
         return out;
     };
 
-    /**
-     * todo for debugging
-     */
-    PRO.dump = function(msg,prec) {
-        var scope = this,
-            txt = [JSON.stringify({
-                len:scope.length,
-                area:scope.areaDeep(),
-                perim:scope.perimeterDeep(),
-                circ:scope.circularityDeep(),
-                inner:(scope.inner ? scope.inner.length : 0),
-                cw:scope.isClockwise(),
-                open:scope.isOpen(),
-                id:scope.id
-            })],
-            out = [], i = 0, p;
-        while (i<scope.points.length) {
-            p = scope.points[i++];
-            out.appendAll([UTIL.round(p.x,prec||5), UTIL.round(p.y,prec||5)]);
-        }
-        txt.push('['+out.join(',')+']');
-        DBUG.log((msg ? msg : '')+txt.join('\n'));
-        if (scope.inner) {
-            scope.inner.forEach(function(p) { p.dump("-- ",prec) });
-        }
-    };
-
     /**
      * return offset polygon(s) from original using distance.  may result in
      * more than one new polygon if trace is self-intersecting or null if new
diff --git a/js/geo-polygons.js b/js/geo-polygons.js
index f5db0bd1..8a15dc8c 100644
--- a/js/geo-polygons.js
+++ b/js/geo-polygons.js
@@ -32,7 +32,6 @@ var gs_base_polygons = exports;
         expand : expand,
         union : union,
         nest : nest,
-        dump : dump,
         diff : doDiff,
         filter : filter,
         toClipper : toClipper,
@@ -107,16 +106,6 @@ var gs_base_polygons = exports;
         return output;
     }
 
-    function dump(poly) {
-        if (Array.isArray(poly)) {
-            poly.forEach(function(p) { dump(p) });
-        } else {
-            console.group({id:poly.id, area:poly.area(), depth:poly.depth, inner:(poly.inner ? poly.inner.length : null)});
-            if (poly.inner) dump(poly.inner);
-            console.groupEnd();
-        };
-    }
-
     /**
      * todo use clipper polytree?
      *
diff --git a/js/kiri-driver-fdm.js b/js/kiri-driver-fdm.js
index 90b74c50..d47abf60 100644
--- a/js/kiri-driver-fdm.js
+++ b/js/kiri-driver-fdm.js
@@ -330,6 +330,7 @@ var gs_kiri_fdm = exports;
                 }
             }
 
+            // exit if no slices
             if (slices.length === 0) break;
 
             // iterate over layer slices, find closest widget, print, eliminate
diff --git a/notes.md b/notes.md
index 87e747af..27eba993 100644
--- a/notes.md
+++ b/notes.md
@@ -7,16 +7,15 @@
 * bail on decimation if it's proving ineffective
 * improve decimation speed by avoiding in/out of Point
 * server-side processing (determine protocol and storage)
-* move more kiri code (like printing) into modules like serial
+* move printing/output into module
 * refactor / simplify POLY.expand (put onus on collector)
-* add simple solid (tube-like) rendering in place of lines
 * cloned objects should share same slice data unless rotated
 * remember object's original position/orientation for reset/multi-object import alignment
 
 # FDM todo
 
-* apply finish speed to exposed top and underside flat areas
 * adaptive column to compensate for fine or layers that finish too quickly and melt
+* apply finish speed to exposed top and underside flat areas
 * store/recover grid:print target against printer selected in device
 * expand internal supporting flats
 * first layer support speed should be same as shell speed
@@ -30,8 +29,6 @@
 * feather sharp tips by reducing extrusion in area of overlap
 * dual extruder support
 * add manual supports
-* fan / layer control
-* update forum
 * option to support interior bridges when 0% infill
 * fix multiple part layout export offset (resend position @ print time)