chore(gis): Clean up API and extract common helpers

modules/gis/src/index.ts

@@ -13,14 +13,17 @@ export {
 } from './lib/geoarrow/geoparquet-metadata';
 export {unpackJSONStringMetadata} from './lib/geoarrow/geoparquet-metadata';
 
-// Utilities
-export {transformBinaryCoords, transformGeoJsonCoords} from './lib/api/transform-coordinates';
+//
+export type {GeojsonGeometryInfo} from './lib/geometry-api/geometry-info';
+export {getGeometryInfo} from './lib/geometry-api/geometry-info';
 
-// Binary Geometries
-// export {flatGeojsonToBinary} from './lib/binary-features/flat-geojson-to-binary';
-// export {geojsonToBinary} from './lib/binary-features/geojson-to-binary';
-// export {geojsonToFlatGeojson} from './lib/binary-features/geojson-to-flat-geojson';
-// export {binaryToGeojson, binaryToGeometry} from './lib/binary-features/binary-to-geojson';
+// Binary Geometry Utilities
+export type {BinaryGeometryInfo} from './lib/binary-geometry-api/binary-geometry-info';
+export {getBinaryGeometryInfo} from './lib/binary-geometry-api/binary-geometry-info';
+export {
+  transformBinaryCoords,
+  transformGeoJsonCoords
+} from './lib/binary-geometry-api/transform-coordinates';
 
 // TABLE CONVERSION
 export {convertGeoArrowToTable} from './lib/table-converters/convert-geoarrow-table';
@@ -48,24 +51,22 @@ export {
 } from './lib/feature-collection-converters/convert-binary-features-to-geojson';
 
 // GEOMETRY ENCODING DETECTION
-export {isTWKB} from './lib/geometry-converters/wkt/convert-twkb-to-geojson';
-export {isWKB} from './lib/geometry-converters/wkt/helpers/parse-wkb-header';
-export {isWKT} from './lib/geometry-converters/wkt/convert-wkt-to-geojson';
+export {isWKB, isTWKB, isWKT} from './lib/geometry-converters/wkb/helpers/parse-wkb-header';
 
-export type {WKBHeader} from './lib/geometry-converters/wkt/helpers/parse-wkb-header';
-export {WKT_MAGIC_STRINGS} from './lib/geometry-converters/wkt/convert-wkt-to-geojson';
+export type {WKBHeader} from './lib/geometry-converters/wkb/helpers/wkb-types';
+export {WKT_MAGIC_STRINGS} from './lib/geometry-converters/wkb/helpers/wkb-types';
 
 // GEOMETRY CONVERSION
 export {convertBinaryGeometryToGeometry} from './lib/geometry-converters/convert-binary-geometry-to-geojson';
 
-export {convertWKTToGeometry} from './lib/geometry-converters/wkt/convert-wkt-to-geojson';
-export {convertWKBToGeometry} from './lib/geometry-converters/wkt/convert-wkb-to-geojson';
-export {convertWKBToBinaryGeometry} from './lib/geometry-converters/wkt/convert-wkb-to-binary-geometry';
-export {convertTWKBToGeometry} from './lib/geometry-converters/wkt/convert-twkb-to-geojson';
+export {convertWKTToGeometry} from './lib/geometry-converters/wkb/convert-wkt-to-geometry';
+export {convertWKBToGeometry} from './lib/geometry-converters/wkb/convert-wkb-to-geometry';
+export {convertWKBToBinaryGeometry} from './lib/geometry-converters/wkb/convert-wkb-to-binary-geometry';
+export {convertTWKBToGeometry} from './lib/geometry-converters/wkb/convert-twkb-to-geometry';
 
-export {convertGeometryToWKT} from './lib/geometry-converters/wkt/convert-geojson-to-wkt';
-export {convertGeometryToWKB} from './lib/geometry-converters/wkt/convert-geojson-to-wkb';
-export {convertGeometryToTWKB} from './lib/geometry-converters/wkt/convert-geojson-to-twkb';
+export {convertGeometryToWKT} from './lib/geometry-converters/wkb/convert-geometry-to-wkt';
+export {convertGeometryToWKB} from './lib/geometry-converters/wkb/convert-geometry-to-wkb';
+export {convertGeometryToTWKB} from './lib/geometry-converters/wkb/convert-geometry-to-twkb';
 
 // CRS
 export type {WKTCRS, ParseWKTCRSOptions} from './lib//wkt-crs/parse-wkt-crs';
@@ -94,5 +95,4 @@ export {updateBoundsFromGeoArrowSamples} from './lib/geoarrow/get-arrow-bounds';
 // EXPERIMENTAL APIs
 
 export {encodeHex, decodeHex} from './lib/utils/hex-transcoder';
-export {extractGeometryInfo as _extractGeometryInfo} from './lib/feature-collection-converters/helpers/extract-geometry-info';
 export {extractNumericPropTypes as _extractNumericPropTypes} from './lib/feature-collection-converters/convert-flat-geojson-to-binary-features';

modules/gis/src/lib/binary-geometry-api/binary-geometry-info.ts

@@ -0,0 +1,75 @@
+// loaders.gl
+// SPDX-License-Identifier: MIT
+// Copyright (c) vis.gl contributors
+
+import type {BinaryGeometry} from '@loaders.gl/schema';
+
+/**
+ * Information about a binary geometry
+ */
+export type BinaryGeometryInfo = {
+  /** The GeoJSON style geometry type corresponding to this particular binary geometry */
+  multiGeometryType:
+    | 'Point'
+    | 'LineString'
+    | 'Polygon'
+    | 'MultiPoint'
+    | 'MultiLineString'
+    | 'MultiPolygon';
+  /** Is this a "Multi" version of the binary geometry? */
+  isMultiGeometry: boolean;
+  /** How many dimensions are the coordinates? */
+  dimension: number;
+  /** How many points does this geometry have? */
+  pointCount: number;
+  /** How many coordinates does this geometry have? */
+  coordinateCount: number;
+};
+
+/**
+ * @returns information about a binary geometry
+ */
+export function getBinaryGeometryInfo(geometry: BinaryGeometry): BinaryGeometryInfo {
+  return {
+    isMultiGeometry: isMultiGeometryType(geometry),
+    multiGeometryType: getMultiGeometryType(geometry),
+    dimension: geometry.positions.size,
+    pointCount: geometry.positions.value.length / geometry.positions.size,
+    coordinateCount: geometry.positions.value.length
+  };
+}
+
+/** @returns true if a binary geometry corresponds to a MultiPoint, MultiLineString or MultiPolygon */
+function isMultiGeometryType(geometry: BinaryGeometry) {
+  switch (geometry.type) {
+    case 'Point':
+      const {positions} = geometry;
+      return positions.value.length / positions.size > 1;
+    case 'LineString':
+      const {pathIndices} = geometry;
+      return pathIndices.value.length > 1;
+    case 'Polygon':
+      const {polygonIndices} = geometry;
+      return polygonIndices.value.length > 1;
+    default:
+      return false;
+  }
+}
+
+/**
+ * @returns geometry type of binary geometry, including MultiPoint, MultiLineString or MultiPolygon
+ */
+function getMultiGeometryType(geometry: BinaryGeometry) {
+  const isMulti = isMultiGeometryType(geometry);
+  switch (geometry.type) {
+    case 'Point':
+      return isMulti ? 'MultiPoint' : 'Point';
+    case 'LineString':
+      return isMulti ? 'MultiLineString' : 'LineString';
+    case 'Polygon':
+      return isMulti ? 'MultiPolygon' : 'Polygon';
+    default:
+      // @ts-expect-error
+      throw new Error(`Illegal geometry type: ${type}`);
+  }
+}

modules/gis/src/lib/api/transform-coordinates.ts → modules/gis/src/lib/binary-geometry-api/transform-coordinates.ts

@@ -38,7 +38,6 @@ function transformBinaryGeometryPositions(binaryGeometry: BinaryGeometry, fn: Tr
 
 /**
  * Apply transformation to every coordinate of GeoJSON features
- *
  * @param  features Array of GeoJSON features
  * @param  fn       Function to call on each coordinate
  * @return          Transformed GeoJSON features

modules/gis/src/lib/feature-collection-converters/convert-flat-geojson-to-binary-features.ts

@@ -12,9 +12,9 @@ import type {
   FlatPoint,
   FlatLineString,
   FlatPolygon,
-  GeojsonGeometryInfo,
   TypedArray
 } from '@loaders.gl/schema';
+import {GeojsonGeometryInfo} from '../geometry-api/geometry-info';
 import {
   PropArrayConstructor,
   Lines,

modules/gis/src/lib/feature-collection-converters/convert-geojson-to-binary-features.ts

@@ -4,7 +4,7 @@
 
 import type {Feature, BinaryFeatureCollection} from '@loaders.gl/schema';
 
-import {extractGeometryInfo} from './helpers/extract-geometry-info';
+import {getGeometryInfo} from '../geometry-api/geometry-info';
 import {convertGeojsonToFlatGeojson} from './convert-geojson-to-flat-geojson';
 import {convertFlatGeojsonToBinaryFeatureCollection} from './convert-flat-geojson-to-binary-features';
 
@@ -29,7 +29,7 @@ export function convertGeojsonToBinaryFeatureCollection(
   features: Feature[],
   options: GeojsonToBinaryOptions = {fixRingWinding: true, triangulate: true}
 ): BinaryFeatureCollection {
-  const geometryInfo = extractGeometryInfo(features);
+  const geometryInfo = getGeometryInfo(features);
   const coordLength = geometryInfo.coordLength;
   const {fixRingWinding} = options;
   const flatFeatures = convertGeojsonToFlatGeojson(features, {coordLength, fixRingWinding});

modules/gis/src/lib/feature-collection-converters/helpers/extract-geometry-info.ts → modules/gis/src/lib/geometry-api/geometry-info.ts

@@ -2,15 +2,29 @@
 // SPDX-License-Identifier: MIT
 // Copyright (c) vis.gl contributors
 
-import type {Feature, GeojsonGeometryInfo} from '@loaders.gl/schema';
+import type {Feature} from '@loaders.gl/schema';
+
+/** Aggregate information for converting GeoJSON into other formats */
+export type GeojsonGeometryInfo = {
+  coordLength: number;
+  pointPositionsCount: number;
+  pointFeaturesCount: number;
+  linePositionsCount: number;
+  linePathsCount: number;
+  lineFeaturesCount: number;
+  polygonPositionsCount: number;
+  polygonObjectsCount: number;
+  polygonRingsCount: number;
+  polygonFeaturesCount: number;
+};
 
 /**
  *  Initial scan over GeoJSON features
  *  Counts number of coordinates of each geometry type and
  *  keeps track of the max coordinate dimensions
  */
 // eslint-disable-next-line complexity, max-statements
-export function extractGeometryInfo(features: Feature[]): GeojsonGeometryInfo {
+export function getGeometryInfo(features: Feature[]): GeojsonGeometryInfo {
   // Counts the number of _positions_, so [x, y, z] counts as one
   let pointPositionsCount = 0;
   let pointFeaturesCount = 0;

modules/gis/src/lib/geometry-converters/convert-geoarrow-to-geojson.ts

@@ -14,8 +14,8 @@ import type {
   Geometry,
   GeoArrowEncoding
 } from '@loaders.gl/schema';
-import {convertWKBToGeometry} from './wkt/convert-wkb-to-geojson';
-import {convertWKTToGeometry} from './wkt/convert-wkt-to-geojson';
+import {convertWKBToGeometry} from './wkb/convert-wkb-to-geometry';
+import {convertWKTToGeometry} from './wkb/convert-wkt-to-geometry';
 
 /**
  * parse geometry from arrow data that is returned from processArrowData()

modules/gis/src/lib/geometry-converters/convert-geometry-to-binary-geometry.ts

@@ -0,0 +1,41 @@
+// loaders.gl
+// SPDX-License-Identifier: MIT
+// Copyright (c) vis.gl contributors
+
+// import type {Feature, BinaryFeatureCollection} from '@loaders.gl/schema';
+
+// import {getGeometryInfo} from '../geometry-api/geometry-info';
+// import {convertGeojsonToFlatGeojson} from './convert-geojson-to-flat-geojson';
+// import {convertFlatGeojsonToBinaryFeatureCollection} from './convert-flat-geojson-to-binary-features';
+
+// /**
+//  * Options for `geojsonToBinary`
+//  */
+// export type GeojsonToBinaryOptions = {
+//   fixRingWinding: boolean;
+//   numericPropKeys?: string[];
+//   PositionDataType?: Float32ArrayConstructor | Float64ArrayConstructor;
+//   triangulate?: boolean;
+// };
+
+// /**
+//  * Convert GeoJSON features to flat binary arrays
+//  *
+//  * @param features
+//  * @param options
+//  * @returns features in binary format, grouped by geometry type
+//  */
+// export function convertGeojsonToBinaryFeatureCollection(
+//   features: Feature[],
+//   options: GeojsonToBinaryOptions = {fixRingWinding: true, triangulate: true}
+// ): BinaryFeatureCollection {
+//   const geometryInfo = getGeometryInfo(features);
+//   const coordLength = geometryInfo.coordLength;
+//   const {fixRingWinding} = options;
+//   const flatFeatures = convertGeojsonToFlatGeojson(features, {coordLength, fixRingWinding});
+//   return convertFlatGeojsonToBinaryFeatureCollection(flatFeatures, geometryInfo, {
+//     numericPropKeys: options.numericPropKeys,
+//     PositionDataType: options.PositionDataType || Float32Array,
+//     triangulate: options.triangulate
+//   });
+// }

modules/gis/src/lib/geometry-converters/convert-to-geojson.ts

@@ -4,10 +4,10 @@
 
 import type {Geometry, BinaryGeometry} from '@loaders.gl/schema';
 
-import {convertGeometryToWKB} from './wkt/convert-geojson-to-wkb';
-import {convertGeometryToWKT} from './wkt/convert-geojson-to-wkt';
-import {convertWKBToBinaryGeometry} from './wkt/convert-wkb-to-binary-geometry';
-import {convertWKTToGeometry} from './wkt/convert-wkt-to-geojson';
+import {convertGeometryToWKB} from './wkb/convert-geometry-to-wkb';
+import {convertGeometryToWKT} from './wkb/convert-geometry-to-wkt';
+import {convertWKBToBinaryGeometry} from './wkb/convert-wkb-to-binary-geometry';
+import {convertWKTToGeometry} from './wkb/convert-wkt-to-geometry';
 
 import {convertBinaryGeometryToGeometry} from './convert-binary-geometry-to-geojson';