@dra2020/baseclient 1.0.35 → 1.0.38

package/lib/poly/mapto.ts

@@ -38,22 +38,35 @@ export function polyMapToByCentroid(districts: G.GeoFeatureCollection, centroids
   let map: any = {};
 
   // Cache district boundboxes for quick containment exclusion
-  let bbDistricts: BB.BoundBox[] = districts.features.map(f => BB.boundbox(f));
+  let fs: G.GeoFeature[] = districts.features;
+  let bbDistricts: BB.BoundBox[] = fs.map(f => BB.boundbox(f));
+  let aDistricts: number[] = fs.map(f => P.polyArea(f));
 
   // Walk over blocks, mapping centroid to district
   Object.keys(centroids).forEach(blockid => {
       let x = centroids[blockid].x;
       let y = centroids[blockid].y;
 
-      let fIn: G.GeoFeature[] = [];
-      districts.features.forEach((fDistrict: G.GeoFeature, i: number) => {
+      let fIn: number[] = [];
+      fs.forEach((fDistrict: G.GeoFeature, i: number) => {
           if (BB.boundboxContains(bbDistricts[i], x, y))
             if (polyContainsPoint(fDistrict, x, y))
-              fIn.push(fDistrict);
+              fIn.push(i);
         });
 
       if (fIn.length == 1)
-        map[blockid] = fIn[0].properties.id;
+        map[blockid] = fs[fIn[0]].properties.id;
+      else if (fIn.length > 1)
+      {
+        // Pick district with smallest area since some times we get malformed content that doesn't
+        // reflect holes in districts when one is nested in another. So theory is smaller district
+        // is a hole in containing larger one(s).
+        let iLow = fIn[0];
+        for (let i = 1; i < fIn.length; i++)
+          if (aDistricts[fIn[i]] < aDistricts[iLow])
+            iLow = fIn[i];
+        map[blockid] = fs[iLow].properties.id;
+      }
     });
 
   return map;

package/lib/poly/poly.ts

@@ -683,6 +683,7 @@ export function featureRewind(poly: any): any
       let d = Util.depthof(poly.geometry.coordinates);
       if (d === 4) closePoly(poly.geometry.coordinates);
       else if (d === 5) poly.geometry.coordinates.forEach(closePoly);
+      poly.geometry.type = d === 4 ? 'Polygon' : 'MultiPolygon';
     }
     return poly;
   }

package/lib/poly/topo.ts

@@ -19,11 +19,16 @@ export type Topo = any;
 
 function getGEOID(f: any): string
 {
-  if (f.features && f.features.length) f = f.features[0];
-  else if (Array.isArray(f)) f = f[0];
+  if (f.features && f.features.length)
+    f = f.features[0];
+  else if (Array.isArray(f))
+    f = f[0];
+  else
+    return '';
   if (f.properties.id !== undefined) return 'id';
   if (f.properties.GEOID !== undefined) return 'GEOID';
   if (f.properties.GEOID10 !== undefined) return 'GEOID10';
+  return '';
 }
 
 export function topoFromCollection(col: any): Topo

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dra2020/baseclient",
-  "version": "1.0.35",
+  "version": "1.0.38",
   "description": "Utility functions for Javascript projects.",
   "main": "dist/baseclient.js",
   "types": "./dist/all/all.d.ts",
@@ -40,6 +40,9 @@
     "webpack-cli": "^4.4.0"
   },
   "dependencies": {
+    "@dra2020/baseclient": "^1.0.36",
+    "@dra2020/baseserver": "^1.0.21",
+    "@dra2020/dra-types": "^1.8.26",
     "@dra2020/topojson-client": "^3.2.7",
     "@dra2020/topojson-server": "^3.0.103",
     "@dra2020/topojson-simplify": "^3.0.102",