From 164c10ed24d44d43f0feb4cebbc43bd2307bbf7d Mon Sep 17 00:00:00 2001 From: Scott Staniewicz Date: Sun, 14 Aug 2022 09:54:02 -0400 Subject: [PATCH] add version-specific gdal instructions --- README.md | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index dd6a0e4..a9e227c 100644 --- a/README.md +++ b/README.md @@ -12,7 +12,11 @@ usage: sardem [-h] [--bbox left bottom right top] [--geojson GEOJSON] [--wkt-fil The default data source, `--data-source NASA`, uses the SRTM 1 arcsecond data. You can also use the newer [Copernicus Digital Surface Model (DSM)](https://registry.opendata.aws/copernicus-dem/). -**Note** To convert the elevation values to heights about the WGS84 ellipsoid (which is the default), GDAL is required. See below: +**Note** To convert the elevation values to heights about the WGS84 ellipsoid (which is the default), or to use the Copernicus data, GDAL is required. +For the Copernicus data, the minimum required GDAL version is 3.4.2; version earlier than 3.4.0 seem to hang upon using `gdalwarp` on the global VRT, and <3.4.2 have an internal bug https://github.com/isce-framework/isce2/issues/556 . + + +See below for installation: ## Setup and installation @@ -44,11 +48,11 @@ pip install sardem GDAL is required for the conversion, which is most easily installed using `conda` (or `mamba`): ```bash -conda install -c conda-forge gdal +conda install -c conda-forge "gdal>=3.4.2" # or # conda install -c conda-forge mamba -# mamba install -c conda-forge gdal +# mamba install -c conda-forge "gdal>=3.4.2" ```