@dra2020/baseclient 1.0.3 → 1.0.6

package/dist/baseclient.js

@@ -1515,6 +1515,16 @@ class GeoMultiCollection {
         }
         return undefined;
     }
+    filter(test) {
+        let m = new GeoMultiCollection();
+        this.forEachEntry(e => {
+            let col = this._col(e);
+            let features = col ? col.features.filter(test) : null;
+            if (features && features.length)
+                m.add(e.tag, null, { type: 'FeatureCollection', features: features }, null);
+        });
+        return m;
+    }
 }
 exports.GeoMultiCollection = GeoMultiCollection;
 var geoIntersectOptions;
@@ -7939,24 +7949,14 @@ function ringsCancel(outerPoly, innerRing) {
 function correctGeometry(f) {
     if (f && f.geometry && f.geometry.type === 'MultiPolygon' && f.geometry.coordinates) {
         let multiPoly = f.geometry.coordinates;
-        /* Comment this out right now - might have been do to not rewinding merge output
-        // topojson will under certain circumstances return a MultiPolygon with the first Polygon containing holes
-        // that are precisely filled by a subsequent polygon. We make a secondary union pass to try to correct for this.
-        // If we have a really degenerate multipolygon (test above some number of polygons) omit this expensive pass
-        // since cleanup is unlikely.
-        if (multiPoly.length > 1 && multiPoly.length < 50)
-        {
-          let result = Q.unionPolys(multiPoly);
-          if (Util.depthof(result) == 4) result = [ result ];
-          multiPoly = result;
-        }
-        */
         // Convert degenerate MultiPolygon to Polygon
         if (multiPoly.length == 1) {
             f.geometry.type = 'Polygon';
             f.geometry.coordinates = multiPoly[0];
         }
     }
+    // TopoJSON does not guarantee proper winding order which messes up later processing. Fix it.
+    P.featureRewind(f);
     return f;
 }
 function topoContiguity(topo) {
@@ -8050,6 +8050,9 @@ function bigTimeString(ms) {
     return `${minutes}:${seconds < 10 ? '0' : ''}${seconds}`;
 }
 const DefaultSimplifyOptions = { minArea: 500 };
+function log(s) {
+    //console.log(s);
+}
 //
 // topoSimplifyCollection:
 //    This implements our simplification strategy for block/precinct level shapes. The basic idea is to
@@ -8072,10 +8075,10 @@ function topoSimplifyCollection(col, options) {
     let elapsedTotal = new Util.Elapsed();
     let elapsed = new Util.Elapsed();
     let topo = topoFromCollection(col);
-    console.log(`topoSimplifyCollection: fromCollection: ${Math.round(elapsed.ms())}ms`);
+    log(`topoSimplifyCollection: fromCollection: ${Math.round(elapsed.ms())}ms`);
     elapsed.start();
     topo = TopoSimplify.presimplify(topo, TopoSimplify['sphericalTriangleArea']);
-    console.log(`topoSimplifyCollection: presimplify: ${Math.round(elapsed.ms())}ms`);
+    log(`topoSimplifyCollection: presimplify: ${Math.round(elapsed.ms())}ms`);
     elapsed.start();
     // Keep iterating on removing simplification from degenerate shapes
     let nTries = 1;
@@ -8131,11 +8134,11 @@ function topoSimplifyCollection(col, options) {
                 }
             }
         });
-        console.log(`topoSimplifyCollection: pass ${nTries}: ${nBad} (${nTiny} tiny) of ${col.features.length} features are degenerate`);
+        log(`topoSimplifyCollection: pass ${nTries}: ${nBad} (${nTiny} tiny) of ${col.features.length} features are degenerate`);
         // If not making progress, keep more points
         if (nBad >= nBadLast) {
             keepweight /= 10;
-            console.log(`topoSimplifyCollection: pass ${nTries}: reducing weight limit to ${keepweight}`);
+            log(`topoSimplifyCollection: pass ${nTries}: reducing weight limit to ${keepweight}`);
         }
         nBadLast = nBad;
         if (nBad && nTries > MAX_TRIES)
@@ -8147,7 +8150,7 @@ function topoSimplifyCollection(col, options) {
         }
         nTries++;
     }
-    console.log(`topoSimplifyCollection: total elapsed time: ${bigTimeString(elapsedTotal.ms())}`);
+    log(`topoSimplifyCollection: total elapsed time: ${bigTimeString(elapsedTotal.ms())}`);
     return col;
 }
 exports.topoSimplifyCollection = topoSimplifyCollection;
@@ -8156,27 +8159,7 @@ function topoMerge(topo, geoids) {
         return null;
     let objects = [];
     geoids.forEach((geoid) => objects.push(topo.objects[geoid]));
-    let f = correctGeometry({ type: 'Feature', properties: {}, geometry: TopoClient.merge(topo, objects) });
-    P.featureRewind(f);
-    /* Comment out for now - may be due to merge output needing to be rewound
-    // If I get a bad output from topoMerge, just do more expensive poly union. This can happen if input polygons
-    // are a little funky (in particular, if they double back along the same edge.
-    if (selfIntersectFast(f))
-    {
-      //console.log('topoMerge: patching selfIntersect');
-      let polys: any[] = [];
-      geoids.forEach((geoid) => polys.push(topoToFeature(topo, geoid).geometry.coordinates));
-      let result = Q.unionPolys(polys);
-      let depth = Util.depthof(result);
-      if (depth === 5 && result.length === 1)
-      {
-        depth = 4;
-        result = result[0];
-      }
-      f = { type: 'feature', properties: {}, geometry: { type: depth == 4 ? 'Polygon' : 'MultiPolygon', coordinates: result } };
-    }
-    */
-    return f;
+    return correctGeometry({ type: 'Feature', properties: {}, geometry: TopoClient.merge(topo, objects) });
 }
 exports.topoMerge = topoMerge;
 function topoMergeFeatures(topo, features) {