search

geotrellis / geotrellis-pointcloud

GeoTrellis PointCloud library to work with any pointcloud data on Spark
Apache License 2.0
26 stars 10 forks source link

Fix the resolutions response from DEM RasterSources #69

Closed jpolchlo closed 4 years ago

jpolchlo commented 4 years ago

While messing around with QGIS via https://github.com/geotrellis/geotrellis-pointcloud-server, I noticed something funny with the DEM raster sources is going on. Observe zooming in on the Red Rocks data:

far near

In the second image, it's clear to see that the pixels appear to have gotten larger, implying that the gridExtent of the requests aren't updating to the new view parameters. It looks from the source like the gridExtent being used everywhere in DEMRasterSource (soon to be TINRasterSource) is simply being pulled from the EPTMetadata. I could be missing something, but the behavior suggests that at least something like this is happening. We need to investigate.

pomadchin commented 4 years ago

Will add some extra context, GeoTrellis PointCloud logs all the requested scenes. During the WCS zoom in operation QGIS always sends the same request:

 /?SERVICE=WCS&VERSION=1.1.1&REQUEST=GetCoverage
...
GRIDOFFSETS=-0.00006233669818305%2C0.00008030763819988

Where GRIDOFFSETS=-0.00006233669818305%2C0.00008030763819988 is the requested resolution.

WMS and WMTS properly handle zoom requests (this can actually be also an explanation why WCS is so much faster).

jpolchlo commented 4 years ago

That's a good observation, and it implies that we need a proper resolution list to be supplied in the GetCapabilites request.

pomadchin commented 4 years ago

https://github.com/geotrellis/geotrellis-pointcloud-server/commit/81b545c7794c998397648bd181acfbb59e28366b 💯

pomadchin commented 4 years ago

image

And all WCS requests work correct now.