@loaders.gl/tile-converter 4.0.0-alpha.9 → 4.0.0-beta.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@loaders.gl/tile-converter",
-  "version": "4.0.0-alpha.9",
+  "version": "4.0.0-beta.1",
   "description": "Converter",
   "license": "MIT",
   "publishConfig": {
@@ -21,7 +21,8 @@
   "sideEffects": false,
   "bin": {
     "tile-converter": "./bin/converter.js",
-    "i3s-server": "./src/i3s-server/bin/www"
+    "slpk-extractor": "./bin/slpk-extractor.js",
+    "i3s-server": "./bin/i3s-server.js"
   },
   "files": [
     "src",
@@ -37,25 +38,25 @@
     "join-images": false
   },
   "scripts": {
-    "pre-build": "npm run build-bundle && npm run build-converter-bundle && npm run build-i3s-attributes-worker && npm run build-3d-tiles-attributes-worker",
+    "pre-build": "npm run build-bundle && npm run build-converter-bundle && npm run build-slpk-extractor-bundle && npm run build-i3s-server-bundle",
     "build-bundle": "esbuild ./src/index.ts --bundle --outfile=dist/dist.min.js --platform=node  --external:join-images",
     "build-converter-bundle": "esbuild src/converter-cli.ts --outfile=dist/converter.min.js --platform=node --target=esnext,node14 --external:join-images --minify --bundle --define:__VERSION__=\\\"$npm_package_version\\\"",
-    "build-i3s-attributes-worker": "esbuild src/workers/i3s-attributes-worker.ts --outfile=dist/i3s-attributes-worker.js --platform=node --target=esnext,node14 --external:join-images --minify --bundle --sourcemap --define:__VERSION__=\\\"$npm_package_version\\\"",
-    "build-3d-tiles-attributes-worker": "esbuild src/workers/3d-tiles-attributes-worker.ts --outfile=dist/3d-tiles-attributes-worker.js --platform=node --target=esnext,node14 --external:join-images --minify --bundle --sourcemap --define:__VERSION__=\\\"$npm_package_version\\\""
+    "build-slpk-extractor-bundle": "esbuild src/slpk-extractor-cli.ts --outfile=dist/slpk-extractor.min.js --platform=node --target=esnext,node14 --external:join-images --minify --bundle --define:__VERSION__=\\\"$npm_package_version\\\"",
+    "build-i3s-server-bundle": "esbuild src/i3s-server/bin/www.ts --outfile=dist/esm/i3s-server/bin/i3s-server.min.js --platform=node --target=esnext,node14 --external:join-images --minify --bundle --define:__VERSION__=\\\"$npm_package_version\\\""
   },
   "dependencies": {
-    "@loaders.gl/3d-tiles": "4.0.0-alpha.9",
-    "@loaders.gl/crypto": "4.0.0-alpha.9",
-    "@loaders.gl/draco": "4.0.0-alpha.9",
-    "@loaders.gl/gltf": "4.0.0-alpha.9",
-    "@loaders.gl/i3s": "4.0.0-alpha.9",
-    "@loaders.gl/images": "4.0.0-alpha.9",
-    "@loaders.gl/loader-utils": "4.0.0-alpha.9",
-    "@loaders.gl/polyfills": "4.0.0-alpha.9",
-    "@loaders.gl/textures": "4.0.0-alpha.9",
-    "@loaders.gl/tiles": "4.0.0-alpha.9",
-    "@loaders.gl/worker-utils": "4.0.0-alpha.9",
-    "@loaders.gl/zip": "4.0.0-alpha.9",
+    "@loaders.gl/3d-tiles": "4.0.0-beta.1",
+    "@loaders.gl/crypto": "4.0.0-beta.1",
+    "@loaders.gl/draco": "4.0.0-beta.1",
+    "@loaders.gl/gltf": "4.0.0-beta.1",
+    "@loaders.gl/i3s": "4.0.0-beta.1",
+    "@loaders.gl/images": "4.0.0-beta.1",
+    "@loaders.gl/loader-utils": "4.0.0-beta.1",
+    "@loaders.gl/polyfills": "4.0.0-beta.1",
+    "@loaders.gl/textures": "4.0.0-beta.1",
+    "@loaders.gl/tiles": "4.0.0-beta.1",
+    "@loaders.gl/worker-utils": "4.0.0-beta.1",
+    "@loaders.gl/zip": "4.0.0-beta.1",
     "@math.gl/core": "^3.5.1",
     "@math.gl/culling": "^3.5.1",
     "@math.gl/geoid": "^3.5.1",
@@ -63,13 +64,14 @@
     "archiver": "^5.0.0",
     "cors": "^2.8.5",
     "crypt": "^0.0.2",
-    "debug": "~2.6.9",
-    "express": "~4.17.3",
+    "debug": "~4.3.4",
+    "express": "~4.18.2",
+    "inquirer": "^8.0.0",
     "json-map-transform": "^1.2.6",
     "jszip": "^3.5.0",
     "md5": "^2.3.0",
     "morgan": "~1.9.1",
-    "uuid": "^8.1.0"
+    "uuid": "^9.0.0"
   },
   "peerDependencies": {
     "@loaders.gl/core": "^4.0.0-alpha.8"
@@ -78,5 +80,9 @@
     "join-images": "^1.1.3",
     "sharp": "^0.31.3"
   },
-  "gitHead": "03ff81ab468f20f3bddeec787aa88d477a7e1c72"
+  "gitHead": "35c625e67132b0784e597d9ddabae8aefea29ff2",
+  "devDependencies": {
+    "@types/express": "^4.17.17",
+    "@types/node": "^20.4.2"
+  }
 }

package/src/3d-tiles-converter/3d-tiles-converter.ts

@@ -1,12 +1,16 @@
-import type {AttributeStorageInfo, FeatureAttribute, NodeReference} from '@loaders.gl/i3s';
+import type {
+  AttributeStorageInfo,
+  FeatureAttribute,
+  NodeReference,
+  I3STilesetHeader
+} from '@loaders.gl/i3s';
 import type {Tiles3DTileJSON} from '@loaders.gl/3d-tiles';
 
 import {join} from 'path';
 import process from 'process';
 import transform from 'json-map-transform';
-import {fetchFile, getLoaderOptions, load, isBrowser} from '@loaders.gl/core';
+import {load, isBrowser} from '@loaders.gl/core';
 import {I3SLoader, I3SAttributeLoader, COORDINATE_SYSTEM} from '@loaders.gl/i3s';
-import {Tileset3D, Tile3D} from '@loaders.gl/tiles';
 import {Geoid} from '@math.gl/geoid';
 
 import {PGMLoader} from '../pgm-loader';
@@ -16,13 +20,12 @@ import {writeFile, removeDir} from '../lib/utils/file-utils';
 import {calculateFilesSize, timeConverter} from '../lib/utils/statistic-utills';
 import {TILESET as tilesetTemplate} from './json-templates/tileset';
 import {createObbFromMbs} from '../i3s-converter/helpers/coordinate-converter';
-import {
-  I3SAttributesData,
-  Tile3dAttributesWorker,
-  transform3DTilesAttributesOnWorker
-} from '../3d-tiles-attributes-worker';
-import {getWorkerURL, WorkerFarm} from '@loaders.gl/worker-utils';
+import {WorkerFarm} from '@loaders.gl/worker-utils';
 import {BROWSER_ERROR_MESSAGE} from '../constants';
+import B3dmConverter, {I3SAttributesData} from './helpers/b3dm-converter';
+import {I3STileHeader} from '@loaders.gl/i3s/src/types';
+import {loadI3SContent} from './helpers/load-i3s';
+import {I3SLoaderOptions} from '@loaders.gl/i3s/src/i3s-loader';
 
 const I3S = 'I3S';
 
@@ -35,9 +38,18 @@ export default class Tiles3DConverter {
   vertexCounter: number;
   conversionStartTime: [number, number];
   geoidHeightModel: Geoid | null;
-  sourceTileset: Tileset3D | null;
-  attributeStorageInfo: AttributeStorageInfo | null;
+  sourceTileset: I3STilesetHeader | null;
+  attributeStorageInfo?: AttributeStorageInfo[] | null;
   workerSource: {[key: string]: string} = {};
+  loaderOptions: I3SLoaderOptions = {
+    _nodeWorkers: true,
+    reuseWorkers: true,
+    i3s: {coordinateSystem: COORDINATE_SYSTEM.LNGLAT_OFFSETS, decodeTextures: false},
+    // We need to load local fs workers because nodejs can't load workers from the Internet
+    'i3s-content': {
+      workerUrl: './modules/i3s/dist/i3s-content-worker-node.js'
+    }
+  };
 
   constructor() {
     this.options = {};
@@ -78,30 +90,19 @@ export default class Tiles3DConverter {
     this.geoidHeightModel = await load(egmFilePath, PGMLoader);
     console.log('Loading egm file completed!'); // eslint-disable-line
 
-    await this.loadWorkers();
-
-    const sourceTilesetJson = await load(inputUrl, I3SLoader, {});
+    this.sourceTileset = await load(inputUrl, I3SLoader, this.loaderOptions);
 
-    this.sourceTileset = new Tileset3D(sourceTilesetJson, {
-      loadOptions: {
-        _nodeWorkers: true,
-        reuseWorkers: true,
-        i3s: {coordinateSystem: COORDINATE_SYSTEM.LNGLAT_OFFSETS, decodeTextures: false}
-        // TODO should no longer be needed with new workers
-        // 'i3s-content-nodejs': {
-        //   workerUrl: './modules/i3s/dist/i3s-content-nodejs-worker.js'
-        // }
-      }
-    });
+    if (!this.sourceTileset) {
+      return;
+    }
 
-    await this.sourceTileset.tilesetInitializationPromise;
-    const rootNode = this.sourceTileset.root!;
-    if (!rootNode.header.obb) {
-      rootNode.header.obb = createObbFromMbs(rootNode.header.mbs);
+    const rootNode = this.sourceTileset?.root;
+    if (!rootNode.obb) {
+      rootNode.obb = createObbFromMbs(rootNode.mbs);
     }
 
     this.tilesetPath = join(`${outputPath}`, `${tilesetName}`);
-    this.attributeStorageInfo = sourceTilesetJson.attributeStorageInfo;
+    this.attributeStorageInfo = this.sourceTileset.attributeStorageInfo;
     // Removing the tilesetPath needed to exclude erroneous files after conversion
     try {
       await removeDir(this.tilesetPath);
@@ -111,7 +112,7 @@ export default class Tiles3DConverter {
 
     const rootTile: Tiles3DTileJSON = {
       boundingVolume: {
-        box: i3sObbTo3dTilesObb(rootNode.header.obb, this.geoidHeightModel)
+        box: i3sObbTo3dTilesObb(rootNode.obb, this.geoidHeightModel)
       },
       geometricError: convertScreenThresholdToGeometricError(rootNode),
       children: []
@@ -137,28 +138,33 @@ export default class Tiles3DConverter {
    * @param childNodeInfo child node to convert
    */
   private async convertChildNode(
-    parentSourceNode: Tile3D,
+    parentSourceNode: I3STileHeader,
     parentNode: Tiles3DTileJSON,
     level: number,
     childNodeInfo: NodeReference
   ): Promise<void> {
     const sourceChild = await this._loadChildNode(parentSourceNode, childNodeInfo);
-    parentSourceNode.children.push(sourceChild);
     if (sourceChild.contentUrl) {
-      await this.sourceTileset!._loadTile(sourceChild);
-      this.vertexCounter += sourceChild.content.vertexCount;
+      const content = await loadI3SContent(this.sourceTileset, sourceChild, this.loaderOptions);
+
+      if (!content) {
+        await this._addChildren(sourceChild, parentNode, level + 1);
+        return;
+      }
+
+      this.vertexCounter += content?.vertexCount || 0;
 
       let featureAttributes: FeatureAttribute | null = null;
       if (this.attributeStorageInfo) {
         featureAttributes = await this._loadChildAttributes(sourceChild, this.attributeStorageInfo);
       }
 
-      if (!sourceChild.header.obb) {
-        sourceChild.header.obb = createObbFromMbs(sourceChild.header.mbs);
+      if (!sourceChild.obb) {
+        sourceChild.obb = createObbFromMbs(sourceChild.mbs);
       }
 
       const boundingVolume = {
-        box: i3sObbTo3dTilesObb(sourceChild.header.obb, this.geoidHeightModel)
+        box: i3sObbTo3dTilesObb(sourceChild.obb, this.geoidHeightModel)
       };
       const child: Tiles3DTileJSON = {
         boundingVolume,
@@ -167,14 +173,13 @@ export default class Tiles3DConverter {
       };
 
       const i3sAttributesData: I3SAttributesData = {
-        tileContent: sourceChild.content,
-        textureFormat: sourceChild?.header?.textureFormat
+        tileContent: content,
+        box: boundingVolume.box,
+        textureFormat: sourceChild.textureFormat
       };
 
-      const b3dm = await transform3DTilesAttributesOnWorker(i3sAttributesData, {
-        source: this.workerSource.tile3dWorkerSource,
-        featureAttributes
-      });
+      const b3dmConverter = new B3dmConverter();
+      const b3dm = await b3dmConverter.convert(i3sAttributesData, featureAttributes);
 
       child.content = {
         uri: `${sourceChild.id}.b3dm`,
@@ -183,7 +188,6 @@ export default class Tiles3DConverter {
       await writeFile(this.tilesetPath, new Uint8Array(b3dm), `${sourceChild.id}.b3dm`);
       parentNode.children.push(child);
 
-      sourceChild.unloadContent();
       await this._addChildren(sourceChild, child, level + 1);
     } else {
       await this._addChildren(sourceChild, parentNode, level + 1);
@@ -197,7 +201,7 @@ export default class Tiles3DConverter {
    * @param level a current level of a tree depth
    */
   private async _addChildren(
-    parentSourceNode: Tile3D,
+    parentSourceNode: I3STileHeader,
     parentNode: Tiles3DTileJSON,
     level: number
   ): Promise<void> {
@@ -205,7 +209,7 @@ export default class Tiles3DConverter {
       return;
     }
     const promises: Promise<void>[] = [];
-    for (const childNodeInfo of parentSourceNode.header.children || []) {
+    for (const childNodeInfo of parentSourceNode.children || []) {
       promises.push(this.convertChildNode(parentSourceNode, parentNode, level, childNodeInfo));
     }
     await Promise.all(promises);
@@ -217,29 +221,31 @@ export default class Tiles3DConverter {
    * @param childNodeInfo child information from 3DNodeIndexDocument
    *   (https://github.com/Esri/i3s-spec/blob/master/docs/1.7/nodeReference.cmn.md)
    */
-  private async _loadChildNode(parentNode: Tile3D, childNodeInfo: NodeReference): Promise<Tile3D> {
+  private async _loadChildNode(
+    parentNode: I3STileHeader,
+    childNodeInfo: NodeReference
+  ): Promise<I3STileHeader> {
     let header;
-    if (this.sourceTileset!.tileset.nodePages) {
+    if (this.sourceTileset?.nodePagesTile) {
       console.log(`Node conversion: ${childNodeInfo.id}`); // eslint-disable-line no-console,no-undef
-      header = await this.sourceTileset!.tileset.nodePagesTile.formTileFromNodePages(
-        childNodeInfo.id
+      header = await this.sourceTileset.nodePagesTile.formTileFromNodePages(
+        parseInt(childNodeInfo.id)
       );
     } else {
-      const {loader} = this.sourceTileset!;
       const nodeUrl = this._relativeUrlToFullUrl(parentNode.url, childNodeInfo.href!);
       // load metadata
       const options = {
         i3s: {
-          ...this.sourceTileset!.loadOptions,
+          ...this.loaderOptions,
           isTileHeader: true,
           loadContent: false
         }
       };
 
       console.log(`Node conversion: ${nodeUrl}`); // eslint-disable-line no-console,no-undef
-      header = await load(nodeUrl, loader, options);
+      header = await load(nodeUrl, I3SLoader, options);
     }
-    return new Tile3D(this.sourceTileset!, header, parentNode);
+    return header;
   }
 
   /**
@@ -247,7 +253,7 @@ export default class Tiles3DConverter {
    * @param baseUrl the base url. A resulting url will be related from this url
    * @param relativeUrl a realtive url of a resource
    */
-  private _relativeUrlToFullUrl(baseUrl: string, relativeUrl: string): string {
+  private _relativeUrlToFullUrl(baseUrl: string = '', relativeUrl: string): string {
     let resultArray = baseUrl.split('/');
     const relativeUrlArray = relativeUrl.split('/');
     for (const folder of relativeUrlArray) {
@@ -271,11 +277,11 @@ export default class Tiles3DConverter {
    * @returns Promise of attributes object.
    */
   private async _loadChildAttributes(
-    sourceChild: Tile3D,
-    attributeStorageInfo: AttributeStorageInfo
+    sourceChild: I3STileHeader,
+    attributeStorageInfo: AttributeStorageInfo[]
   ): Promise<FeatureAttribute> {
     const promises: any[] = [];
-    const {attributeUrls} = sourceChild.header;
+    const {attributeUrls = []} = sourceChild;
 
     for (let index = 0; index < attributeUrls.length; index++) {
       const inputUrl = attributeUrls[index];
@@ -342,14 +348,4 @@ export default class Tiles3DConverter {
     console.log(`File(s) size: `, filesSize, ' bytes'); // eslint-disable-line
     console.log(`------------------------------------------------`); // eslint-disable-line
   }
-
-  private async loadWorkers(): Promise<void> {
-    console.log(`Loading workers source...`); // eslint-disable-line no-undef, no-console
-    const tile3dAttributesWorkerUrl = getWorkerURL(Tile3dAttributesWorker, {...getLoaderOptions()});
-    const sourceResponse = await fetchFile(tile3dAttributesWorkerUrl);
-    const source = await sourceResponse.text();
-
-    this.workerSource.tile3dWorkerSource = source;
-    console.log(`Loading workers source completed!`); // eslint-disable-line no-undef, no-console
-  }
 }

package/src/3d-tiles-converter/helpers/b3dm-converter.ts

@@ -1,5 +1,4 @@
-import type {I3SAttributesData} from '../../3d-tiles-attributes-worker';
-
+import type {I3STileContent} from '@loaders.gl/i3s';
 import {encodeSync} from '@loaders.gl/core';
 import {GLTFScenegraph, GLTFWriter} from '@loaders.gl/gltf';
 import {Tile3DWriter} from '@loaders.gl/3d-tiles';
@@ -11,6 +10,12 @@ import {generateSyntheticIndices} from '../../lib/utils/geometry-utils';
 const Z_UP_TO_Y_UP_MATRIX = new Matrix4([1, 0, 0, 0, 0, 0, -1, 0, 0, 1, 0, 0, 0, 0, 0, 1]);
 const scratchVector = new Vector3();
 
+export type I3SAttributesData = {
+  tileContent: I3STileContent;
+  box: number[];
+  textureFormat: string;
+};
+
 /**
  * Converts content of an I3S node to *.b3dm's file content
  */
@@ -28,7 +33,7 @@ export default class B3dmConverter {
     i3sAttributesData: I3SAttributesData,
     featureAttributes: any = null
   ): Promise<ArrayBuffer> {
-    const gltf = await this.buildGltf(i3sAttributesData, featureAttributes);
+    const gltf = await this.buildGLTF(i3sAttributesData, featureAttributes);
     const b3dm = encodeSync(
       {
         gltfEncoded: new Uint8Array(gltf),
@@ -46,23 +51,16 @@ export default class B3dmConverter {
    * @param i3sTile - Tile3D instance for I3S node
    * @returns - encoded glb content
    */
-  async buildGltf(
+  async buildGLTF(
     i3sAttributesData: I3SAttributesData,
     featureAttributes: any
   ): Promise<ArrayBuffer> {
-    const {tileContent, textureFormat} = i3sAttributesData;
-    const {
-      material,
-      attributes,
-      indices: originalIndices,
-      cartesianOrigin,
-      cartographicOrigin,
-      modelMatrix
-    } = tileContent;
+    const {tileContent, textureFormat, box} = i3sAttributesData;
+    const {material, attributes, indices: originalIndices, modelMatrix} = tileContent;
     const gltfBuilder = new GLTFScenegraph();
 
-    const textureIndex = await this._addI3sTextureToGltf(tileContent, textureFormat, gltfBuilder);
-    const pbrMaterialInfo = this._convertI3sMaterialToGltfMaterial(material, textureIndex);
+    const textureIndex = await this._addI3sTextureToGLTF(tileContent, textureFormat, gltfBuilder);
+    const pbrMaterialInfo = this._convertI3sMaterialToGLTFMaterial(material, textureIndex);
     const materialIndex = gltfBuilder.addMaterial(pbrMaterialInfo);
 
     const positions = attributes.positions;
@@ -75,6 +73,12 @@ export default class B3dmConverter {
       );
     }
 
+    const cartesianOrigin = new Vector3(box);
+    const cartographicOrigin = Ellipsoid.WGS84.cartesianToCartographic(
+      cartesianOrigin,
+      new Vector3()
+    );
+
     attributes.positions.value = this._normalizePositions(
       positionsValue,
       cartesianOrigin,
@@ -111,7 +115,7 @@ export default class B3dmConverter {
    * @param {GLTFScenegraph} gltfBuilder - gltfScenegraph instance to construct GLTF
    * @returns {Promise<number | null>} - GLTF texture index
    */
-  async _addI3sTextureToGltf(tileContent, textureFormat, gltfBuilder) {
+  async _addI3sTextureToGLTF(tileContent, textureFormat, gltfBuilder) {
     const {texture, material, attributes} = tileContent;
     let textureIndex = null;
     let selectedTexture = texture;
@@ -218,7 +222,7 @@ export default class B3dmConverter {
    * @param {number | null} textureIndex - texture index in GLTF
    * @returns {object} GLTF material
    */
-  _convertI3sMaterialToGltfMaterial(material, textureIndex) {
+  _convertI3sMaterialToGLTFMaterial(material, textureIndex) {
     const isTextureIndexExists = textureIndex !== null;
 
     if (!material) {
@@ -244,7 +248,7 @@ export default class B3dmConverter {
     }
 
     if (textureIndex !== null) {
-      material = this._setGltfTexture(material, textureIndex);
+      material = this._setGLTFTexture(material, textureIndex);
     }
 
     return material;
@@ -256,7 +260,7 @@ export default class B3dmConverter {
    * @param {number} textureIndex - texture index in GLTF
    * @returns {void}
    */
-  _setGltfTexture(materialDefinition, textureIndex) {
+  _setGLTFTexture(materialDefinition, textureIndex) {
     const material = {
       ...materialDefinition,
       pbrMetallicRoughness: {...materialDefinition.pbrMetallicRoughness}

package/src/3d-tiles-converter/helpers/load-i3s.ts

@@ -0,0 +1,52 @@
+import {load} from '@loaders.gl/core';
+import {
+  I3STileContent,
+  I3STileHeader,
+  I3STilesetHeader,
+  I3SLoader,
+  I3SLoaderOptions
+} from '@loaders.gl/i3s';
+
+/**
+ * Load I3S node content
+ * @param sourceTileset - source layer JSON
+ * @param sourceTile - source I3S node metadata
+ * @param tilesetLoadOptions - load options for Tiles3DLoader
+ * @returns - 3DTiles tile content or null
+ */
+export const loadI3SContent = async (
+  sourceTileset: I3STilesetHeader | null,
+  sourceTile: I3STileHeader,
+  tilesetLoadOptions: I3SLoaderOptions
+): Promise<I3STileContent | null> => {
+  if (!sourceTileset || !sourceTile.contentUrl) {
+    return null;
+  }
+
+  const loadOptions = {
+    ...tilesetLoadOptions,
+    i3s: {
+      ...tilesetLoadOptions.i3s,
+      isTileset: false,
+      isTileHeader: false,
+      _tileOptions: {
+        attributeUrls: sourceTile.attributeUrls,
+        textureUrl: sourceTile.textureUrl,
+        textureFormat: sourceTile.textureFormat,
+        textureLoaderOptions: sourceTile.textureLoaderOptions,
+        materialDefinition: sourceTile.materialDefinition,
+        isDracoGeometry: sourceTile.isDracoGeometry,
+        mbs: sourceTile.mbs
+      },
+      _tilesetOptions: {
+        store: sourceTileset.store,
+        attributeStorageInfo: sourceTileset.attributeStorageInfo,
+        fields: sourceTileset.fields
+      }
+    }
+  };
+  const tileContent = await load(sourceTile.contentUrl, I3SLoader, loadOptions);
+
+  // @ts-expect-error
+  return tileContent;
+};

package/src/3d-tiles-converter/helpers/texture-atlas.ts

@@ -1,3 +1,5 @@
+import type {TypedArray} from '@loaders.gl/loader-utils';
+
 /**
  * Apply uvRegions to texture coordinates.
  * Spec - https://github.com/Esri/i3s-spec/blob/master/docs/1.7/geometryUVRegion.cmn.md
@@ -5,7 +7,7 @@
  * @param texCoords
  * @param uvRegions
  */
-export function convertTextureAtlas(texCoords: Float32Array, uvRegions: Uint16Array): Float32Array {
+export function convertTextureAtlas(texCoords: TypedArray, uvRegions: TypedArray): Float32Array {
   const convertedTexCoords = new Float32Array(texCoords.length);
   const normalisedRegions = normalizeRegions(uvRegions);
 
@@ -43,7 +45,9 @@ function fract(uv: [number, number]): [number, number] {
  * Normalize uvRegions by dividing by the maximum Uint16 value
  * @param regions
  */
-function normalizeRegions(regions: Uint16Array): number[] {
+function normalizeRegions(regions: TypedArray): number[] {
+  // The code is for Uint16Array because it is the spec requirement
+  // https://github.com/Esri/i3s-spec/blob/master/docs/1.8/geometryUVRegion.cmn.md
   const MAX_UINT_16_VALUE = 65535;
   const normalizedRegions: number[] = [];