refactor: remove optional zoom params from queryElevations

Author: EdwardO

src/source/raster_dem_tile_source.ts

@@ -192,23 +192,14 @@ export class RasterDEMTileSource extends RasterTileSource implements Source {
      *
      * Returns `null` for coordinates where no DEM tile is currently loaded.
      *
-     * Tiles are not evicted during a single call, so it is safe to batch-process
-     * a large number of coordinates (e.g. a full GPX track) in one call.
-     *
      * @param lnglats - Array of geographic coordinates to query.
-     * @param minzoom - Minimum zoom to consider (default: source minzoom). Tiles below this are skipped.
-     * @param maxzoom - Maximum zoom to consider (default: source maxzoom). Search starts here.
      * @returns Array of elevation results (or `null` where no data is available),
      *   in the same order as the input coordinates.
      *
      * @example
      * ```ts
      * const demSource = map.getSource('terrain-dem') as RasterDEMTileSource;
-     * // Query with minimum zoom 12 to avoid low-resolution data
-     * const results = demSource.queryElevations(
-     *     [[7.0, 45.0], [7.1, 45.1]],
-     *     12 // minzoom
-     * );
+     * const results = demSource.queryElevations([[7.0, 45.0], [7.1, 45.1]]);
      * const missing = results.filter(r => r === null).length;
      * if (missing > 0) console.log(`${missing} points have no loaded tile`);
      * results.forEach((r, i) => {
@@ -217,9 +208,7 @@ export class RasterDEMTileSource extends RasterTileSource implements Source {
      * ```
      */
     queryElevations(
-        lnglats: LngLatLike[],
-        minzoom?: number,
-        maxzoom?: number
+        lnglats: LngLatLike[]
     ): (ElevationQueryResult | null)[] {
         if (!this.map) {
             throw new Error('Source is not added to a map');
@@ -230,16 +219,13 @@ export class RasterDEMTileSource extends RasterTileSource implements Source {
             throw new Error(`No tile manager found for source "${this.id}"`);
         }
 
-        const maxZ = Math.min(maxzoom ?? this.maxzoom, this.maxzoom);
-        const minZ = Math.max(minzoom ?? this.minzoom, this.minzoom);
-
         return lnglats.map(ll => {
             const lnglat = LngLat.convert(ll).wrap();
             const mercator = MercatorCoordinate.fromLngLat(lnglat);
             const mx = Math.max(0, Math.min(1 - 1e-15, mercator.x));
             const my = Math.max(0, Math.min(1 - 1e-15, mercator.y));
 
-            for (let z = maxZ; z >= minZ; z--) {
+            for (let z = this.maxzoom; z >= this.minzoom; z--) {
                 const tileCount = 1 << z;
                 const tileX = Math.min(Math.floor(mx * tileCount), tileCount - 1);
                 const tileY = Math.min(Math.floor(my * tileCount), tileCount - 1);

src/source/raster_dem_tile_source_query.test.ts

@@ -158,47 +158,6 @@ describe('RasterDEMTileSource#queryElevations', () => {
         expect(results[0].tileZoom).toBe(2);
     });
 
-    test('respects minzoom option to skip low-res tiles', () => {
-        const demZ0 = createUniformDEM(4, 50);
-        const demZ2 = createUniformDEM(4, 200);
-
-        const source = createMockSource({
-            tiles: [
-                {z: 0, x: 0, y: 0, dem: demZ0},
-                {z: 2, x: 0, y: 0, dem: demZ2}
-            ],
-            maxzoom: 5
-        });
-
-        // With minzoom: 2, should find the z2 tile
-        const results = source.queryElevations([[-135, 80]], 2);
-        expect(results[0]).not.toBeNull();
-        expect(results[0].tileZoom).toBe(2);
-
-        // With minzoom: 3, neither z0 nor z2 qualifies → null
-        const results2 = source.queryElevations([[-135, 80]], 3);
-        expect(results2[0]).toBeNull();
-    });
-
-    test('respects maxzoom option to limit resolution', () => {
-        const demZ0 = createUniformDEM(4, 50);
-        const demZ2 = createUniformDEM(4, 200);
-
-        const source = createMockSource({
-            tiles: [
-                {z: 0, x: 0, y: 0, dem: demZ0},
-                {z: 2, x: 0, y: 0, dem: demZ2}
-            ],
-            maxzoom: 5
-        });
-
-        // With maxzoom: 1, should skip z2 and fall back to z0
-        const results = source.queryElevations([[-135, 80]], undefined, 1);
-        expect(results[0]).not.toBeNull();
-        expect(results[0].elevation).toBeCloseTo(50, 0);
-        expect(results[0].tileZoom).toBe(0);
-    });
-
     test('falls back to lower-zoom tile when higher zoom not loaded', () => {
         const demZ0 = createUniformDEM(4, 100);