feat: Add AbortSignal support to GeoTIFF.fetchTile (#243)

Author: kylebarron

packages/deck.gl-geotiff/package.json

@@ -46,7 +46,7 @@
     "vitest": "^4.0.18"
   },
   "dependencies": {
-    "@cogeotiff/core": "^9.1.2",
+    "@cogeotiff/core": "^9.2.0",
     "@developmentseed/affine": "workspace:^",
     "@developmentseed/deck.gl-raster": "workspace:^",
     "@developmentseed/geotiff": "workspace:^",

packages/deck.gl-geotiff/src/geotiff/render-pipeline.ts

@@ -126,9 +126,8 @@ function createUnormPipeline(
     image: GeoTIFF | Overview,
     options: GetTileDataOptions,
   ) => {
-    const { device, x, y } = options;
-    // TODO: pass down signal
-    const tile = await image.fetchTile(x, y);
+    const { device, x, y, signal } = options;
+    const tile = await image.fetchTile(x, y, { signal });
     let { array } = tile;
 
     let numSamples = samplesPerPixel;

packages/geotiff/package.json

@@ -50,7 +50,7 @@
     "@chunkd/source": "^11.1.0",
     "@chunkd/source-http": "^11.1.1",
     "@chunkd/source-memory": "^11.0.2",
-    "@cogeotiff/core": "^9.1.2",
+    "@cogeotiff/core": "^9.2.0",
     "@developmentseed/affine": "workspace:^",
     "@developmentseed/morecantile": "workspace:^",
     "uuid": "^13.0.0"

packages/geotiff/src/fetch.ts

@@ -27,18 +27,17 @@ interface HasTiffReference extends HasTransform {
   readonly nodata: number | null;
 }
 
-// TODO: add AbortSignal support.
-// https://github.com/blacha/cogeotiff/issues/1397
 export async function fetchTile(
   self: HasTiffReference,
   x: number,
   y: number,
+  options: { signal?: AbortSignal } = {},
 ): Promise<Tile> {
   if (self.maskImage != null) {
     throw new Error("Mask fetching not implemented yet");
   }
 
-  const tile = await self.image.getTile(x, y);
+  const tile = await self.image.getTile(x, y, options);
   if (tile === null) {
     throw new Error("Tile not found");
   }

packages/geotiff/src/geotiff.ts

@@ -264,10 +264,12 @@ export class GeoTIFF {
   // Mixins
 
   /** Fetch a single tile from the full-resolution image. */
-  // TODO: support AbortSignal
-  // https://github.com/blacha/cogeotiff/issues/1397
-  async fetchTile(x: number, y: number): Promise<Tile> {
-    return await fetchTile(this, x, y);
+  async fetchTile(
+    x: number,
+    y: number,
+    options: { signal?: AbortSignal } = {},
+  ): Promise<Tile> {
+    return await fetchTile(this, x, y, options);
   }
 
   // Transform mixin

packages/geotiff/src/overview.ts

@@ -70,10 +70,12 @@ export class Overview {
   }
 
   /** Fetch a single tile from the full-resolution image. */
-  // TODO: support AbortSignal
-  // https://github.com/blacha/cogeotiff/issues/1397
-  async fetchTile(x: number, y: number): Promise<Tile> {
-    return await fetchTile(this, x, y);
+  async fetchTile(
+    x: number,
+    y: number,
+    options: { signal?: AbortSignal } = {},
+  ): Promise<Tile> {
+    return await fetchTile(this, x, y, options);
   }
 
   /**