Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation of vertical reference required for DEM #11

Open
S0sh0rt opened this issue Mar 23, 2022 · 2 comments
Open

Documentation of vertical reference required for DEM #11

S0sh0rt opened this issue Mar 23, 2022 · 2 comments
Labels
documentation Improvements or additions to documentation

Comments

@S0sh0rt
Copy link

S0sh0rt commented Mar 23, 2022

What is the vertical reference for the DEMs? Most DEMs are MSL, but the SAR computations require WGS84. Are you applying EGM 96 or 2008 to the DEM?

@alexamici
Copy link
Member

alexamici commented Mar 23, 2022

This is entirely left to GDAL/Proj, if the metadata on the DEM files is correct and Proj handles the translation to geocent / ECEF, then you are fine.

You can see in the download script for the notebooks that I add the explicit vertical reference EPSG:5773 to the command line in the pre-process step, as the DEM uses EGM96 heights but doesn't declare it: gdalwarp -r bilinear -s_srs EPSG:4326+5773 ... Then GDAL converts the DEM to WGS84 ellipsoid heights and add correct metadata.

@alexamici
Copy link
Member

@S0sh0rt but you are right, I need to document this. [I was quite sure I mentioned it in the README, but it's not in the there]

@alexamici alexamici added the documentation Improvements or additions to documentation label Mar 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants