-
Notifications
You must be signed in to change notification settings - Fork 2
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
Account for vertical datum offsets #2
Comments
I agree with this idea. I think we can create a dict as you mention, and then let the user specify all DEM types with *_E extensions. We can then have a Let me know if this sounds like a plan, |
Sounds good! Would also be good to verify that we can reproduce the OpenTopography _E products using our internal PROJ geoid offset corrections. There may be some discrepancies between how OT prepared the _E heights vs. how we're doing it. In general, this process of overriding the default CRS in the geotiff header for DEMs lacking 3D CRS is something we are going to have to do over and over again. There are some examples in SlideRule repo for 3DEP and PGC ArcticDEM/REMA products. So we can prototype some frameworks here for OT examples using a dict, but good to think about options for a more general implementation that can be used by other packages. Can discuss. |
On your first point, they are not exactly one to one, see the left plot on code line 52 of this notebook. From the products currently available on open topography, the following have ellipsoidal heights available directly from API itself:
These products are referenced with respect to the following geoid model and do not have existing ellipsoidal versions:
The For now we can provide the option to return ellipsoidal versions of the DEMs listed in second block. I am thinking we leave it to WGS84 ellipsoid for now, and if the users want a particular version of horizontal CRS, they can provide EPSG code for the horizontal part. This should be a good start, as I think epoch-aware CRS transformations will take some time for the community to fully digest, and we can revisit that once documentation is solid, with proper WKT2 strings. What do you think @dshean? I will work on adding these functions then today 😊 |
Seems like a reasonable plan. I would not exclude SRTM15Plus - while it may include water, it has land elevations as well. Should be the same CRS as SRTMGL3 products. I think it could be good to explicitly set compound CRS with vertical EPSG:4979 for the WGS84 ellipsoid height output datasets (and definitions for input datasets that do not require vertical offset). I think most of the output DEMs are in local UTM zone? I think most of these inputs are using WGS84 (or GRS80 for EPSG: 3035), so don't need to worry as much about different ellipsoid models and the need to define a specific WGS84 realization to enable proper 3D transformations. |
The latest code has three major improvements:
|
Back from my walk, I have a question @dshean, how do we correctly specify input_crs for EU_DTM? Maybe I am missing something simple, so all input is appreciated! |
Yeah, the compound CRS need to use the same ellipsoid. Even though latest definitions of GRS80 and WGS84 are very similar (within mm), they're not exactly the same. The issue here seems to be the t_srs EPSG:3035+EPSG:7912.
I tried combinations of other EPSG for ellipsoid heights, but non worked out of the box. I think |
This is my first time working with the ETRF ensemble datum. Looks like similar issues for WGS84 Ensemble, only the ETRF has much better ensemble accuracy (0.1 m as opposed to 2.0 m) |
Can create a dictionary for each available global DEM dataset and the required vertical datum transformation. For example COP30 requires EGM2008 to WGS84 offset grid: https://github.com/dshean/demcoreg/blob/master/demcoreg/get_refdem.sh#L47
Some of the available Global DEM datasets are already corrected for ellipsoidal heights, specified with the '_E' suffix (e.g. 'SRTMGL1_E')
The text was updated successfully, but these errors were encountered: