-
Notifications
You must be signed in to change notification settings - Fork 797
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
createOperations(): do Helmert transformation in 2D when one of source or target CRS is compound #4337
Conversation
…e or target CRS is compound Fixes OSGeo#4335
Sorry, shouldn't the additional || condition here only pin if they are both Compound CRSs and have the same vertical CRS? E.g. one might be AHD (EPSG:5711), and the other AVWS (EPSG:9458). |
I was wondering too, and I'm not sure to be honest. There's a bit of ambiguity related to Helmert transformations in the EPSG dataset if they are intended to be applied in 2D only or 3D. There are different method codes in theory but the "(geog2D domain)" ones are always used, even in cases when they are 3D. So it seems safer to do 3D only when both sides are Geographic3D crs. If I do,
The push/pop +v_3 here seems at least consistent with the below pipeline which uses the horizontal grid for GDA94 to GDA2020:
|
I'm a bit uncertain on that too. I know for the Helmert transformation from GDA94 to GDA2020 we get a nation-wide 9cm vertical offset between the datums (though in this case we are just talking about horizontal datums, but their 3D equivalents do have height difference as the ITRF to which they are pinned is moving in a way that local heights change). I don't know if proj offers any pathways of helmert transforms between vertical datums though. I suspect it does, though they would be rare, e.g. the tunnel datum for the UK is 100 m below the spheroid. |
yes, looking at the GDA2020 technical manual (https://www.icsm.gov.au/sites/default/files/GDA2020%20Technical%20Manual%20V1.8_published_0.pdf , e.g. example page 29), ellipsoidal height change seems the thing to do. Fine tuning for this transformation in #4341 |
Fixes #4335