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

Support for SRTMv4 data #9

Open
themohitkhare opened this issue Dec 27, 2020 · 7 comments
Open

Support for SRTMv4 data #9

themohitkhare opened this issue Dec 27, 2020 · 7 comments

Comments

@themohitkhare
Copy link

themohitkhare commented Dec 27, 2020

They have released newer SRTM data, Is it possible to add local data support?
https://cgiarcsi.community/data/srtm-90m-digital-elevation-database-v4-1/

@xivk
Copy link
Contributor

xivk commented Jan 7, 2021

This is definitely possible but I personally don't have the time right now. Pull-requests welcome! 👍

@themohitkhare
Copy link
Author

themohitkhare commented Jan 10, 2021

I was thinking the same thing, but they have changed the file format for the version 4. Need some help in how can I parse this latest format.

@xivk
Copy link
Contributor

xivk commented Jan 19, 2021

Is the current way of doing things broken because of some hardcoded URL? We host the source data on our own server so we don't notice any changes to the source data.

@xivk
Copy link
Contributor

xivk commented Jan 19, 2021

If you have some code example somewhere that can parse this new format, please post it here (doesn't have to be C#).

@themohitkhare
Copy link
Author

The data is ACSII file, with a 2d array holding the elevation data. But It would take some time to understand as it is all new to me. I believe the tiles contains latitude and longitude data.

https://drive.google.com/drive/folders/17dnXkQKlF_fcqqETrHco5cVfF3R7kty0

@xfischer
Copy link

Hi ! The data is in Esri Ascii Grid format, well documented. (see here)
I have coded a reader here if you are interested : https://github.com/dem-net/DEM.Net/blob/46c8546117d51b3485912e9f1568587760dfa2c4/DEM.Net.Core/IO/Raster/AsciiGridFile.cs

@themohitkhare
Copy link
Author

I tried the above reader, but it seems like the SRTM class is tightly coupled with the .hgt type used in SRTMv3. Either I would have to change the entire structure, or @xivk can add better structure allowing easy compatibility with SRTMv4 ACSII structure.
Maybe further abstraction is the way to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants