Skip to content
This repository was archived by the owner on May 18, 2022. It is now read-only.

fix #82 datumgrid added for Hungarian national projection elevation system#97

Open
OSGeoLabBp wants to merge 3 commits intoOSGeo:masterfrom
OSGeoLabBp:master
Open

fix #82 datumgrid added for Hungarian national projection elevation system#97
OSGeoLabBp wants to merge 3 commits intoOSGeo:masterfrom
OSGeoLabBp:master

Conversation

@OSGeoLabBp
Copy link

No description provided.

Copy link
Member

@rouault rouault left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Has EPSG been approached to register those 2 grids ?

* ISL No newline at end of file
* ISL

### Hungary: ETRS89 HD72/EOV (EPSG:23700)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you move that before the "## Included init-files" section ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was no approache to EPSG so far.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was no approache to EPSG so far.

As underlined in CONTRIBUTING.md, that would be very good to do so, otherwise PROJ has to act as a 'geodetic registry' to create custom database records that register the transformation grids. EPSG has much more experience than us to do that and in a consistent way

*Format*: NTv2
*License*: [Creative Commons Attribution 4.0](https://creativecommons.org/licenses/by/4.0/)

Grid transformation from ETRS89 to HD72/EOV.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • is it offsets from ETRS89 to HD72 ? This is slightly unusual, and against the EPSG practice to go from "local" to "global" CRS
  • hum, HD72/EOV is a projected CRS.... Don't you mean the geographic 2D CRS EPSG:4237 instead ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tried to compare with the Helmert-based transformation included in EPSG, EPSG:1449, HD72 to ETRS89 (2)

$ echo "20 47 0" | src/cct -d 8 +proj=pipeline +proj=unitconvert +xy_in=deg +xy_out=rad +step +proj=push +v_3 +step +proj=cart +ellps=GRS67 +step +proj=helmert +x=52.684 +y=-71.194 +z=-13.975 +rx=0.312 +ry=0.1063 +rz=0.3729 +s=1.0191 +convention=coordinate_frame +step +inv +proj=cart +ellps=GRS80 +step +proj=pop +v_3 +step +proj=unitconvert +xy_in=rad +xy_out=deg
  19.99887740    46.99973951    0.00000000           inf

I do get a compatible result when using

$ echo "20 47 0"  | src/cct -d 8  +proj=pipeline +step +proj=hgridshift +grids=etrs2eov_notowgs.gsb
   19.99887893     46.99973967    0.00000000           inf

So this means that this grid is from HD72 to ETRS89. It should likely be renamed so that it is obvious

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The offsets are between EPSG:23700 and EPSG_4258. The HD72/EOV refers to both the datum and projection system. I don't why this name is used. The cs2cs command is:
cs2cs -f "%.7f" +init=epsg:23700 +nadgrids=etrs2eov_notowgs.gsb +geoidgrids=geoid_eht2014.gtx +to +init=epsg:4258

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

he offsets are between EPSG:23700 and EPSG_4258

That doesn't make sense from a modelization point of view. A NTv2 grid is the correction between 2 geographic CRS. Of course it can be (indirectly) applied to a projected CRS based on them, such as EPSG:23700 is based upon EPSG:4237, but from a documentation point of view, only the geographic CRS must be documented. That is this grid is from EPSG:4237 (HD72) to EPSG:4258 (ETRS89)

filelist.csv Outdated
g2018u0.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,,EPSG:6319,EPSG:5703,,us_noaa,NOAA,Public Domain
g2018p0.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,,EPSG:6319,EPSG:6641,,us_noaa,NOAA,Public Domain
etrs2eov_notowgs.gsb,HORIZONTAL_OFFSET,,EPSG:4258,EPSG:23700,,BME-GEOD,www.geod.bme.hu,Creative Commons Attribution 4.0 https://creativecommons.org/licenses/by/4.0/
geoid_eht2014.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,EPSG:4258,EPSG:23700,EPSG:4258,,BME-GEOD,www.geod.bme.hu,Creative Commons Attribution 4.0 https://creativecommons.org/licenses/by/4.0/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrong EPSG codes for geoid_eht2014.gtx. Source should be EPSG:4937 (ETRS89 Geographic 3D CRS) and target I'm not sure, either EPSG:5705 (Baltic 1977 height) or EPSG:5786 (Baltic 1982 height) or 'EPSG:8357 (Baltic 1957 height)


* etrs2eov_notowgs.gsb

### Hungary: ETRS89 heights -> Baltic heights
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

which of the baltic heights ? EPSG references 3 versions: EPSG:5705 (Baltic 1977 height) or EPSG:5786 (Baltic 1982 height) or 'EPSG:8357 (Baltic 1957 height)

filelist.csv Outdated
g2012bp0.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,,EPSG:6319,EPSG:6641,,us_noaa,NOAA,Public Domain
g2018u0.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,,EPSG:6319,EPSG:5703,,us_noaa,NOAA,Public Domain
g2018p0.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,,EPSG:6319,EPSG:6641,,us_noaa,NOAA,Public Domain
etrs2eov_notowgs.gsb,HORIZONTAL_OFFSET,,EPSG:4258,EPSG:23700,,BME-GEOD,www.geod.bme.hu,Creative Commons Attribution 4.0 https://creativecommons.org/licenses/by/4.0/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EPSG:23700 shoud likely be EPSG:4237 instead .

For the agency_name colum, can you use hu_bme_geod ? for that record and the next one

@rouault
Copy link
Member

rouault commented Dec 18, 2019

and could you rebase against latest master ?

ISN_vel_beta.ct2
ISN_vel_beta.gtx

### Hungary: HD72/EOV (EPSG:23700) ETRS89
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Hungary: HD72/EOV (EPSG:23700) ETRS89
### Hungary: HD72 (EPSG:4237) ETRS89

*Format*: NTv2
*License*: [Creative Commons Attribution 4.0](https://creativecommons.org/licenses/by/4.0/)

Grid transformation from HD72/EOV to ETRS89.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Grid transformation from HD72/EOV to ETRS89.
Grid transformation from HD72 to ETRS89 (typically applied for transformations from projected CRS HD72/EOV (EPSG:23700) to ETRS89)

g2018u0.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,,EPSG:6319,EPSG:5703,,us_noaa,NOAA,Public Domain
g2018p0.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,,EPSG:6319,EPSG:6641,,us_noaa,NOAA,Public Domain
etrs2eov_notowgs.gsb,HORIZONTAL_OFFSET,,EPSG:4258,EPSG:8357,,hu_bme_geod,www.geod.bme.hu,Creative Commons Attribution 4.0 https://creativecommons.org/licenses/by/4.0/
geoid_eht2014.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,EPSG:4258,EPSG:23700,EPSG:4258,,hu_bme_geod,www.geod.bme.hu,Creative Commons Attribution 4.0 https://creativecommons.org/licenses/by/4.0/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
geoid_eht2014.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,EPSG:4258,EPSG:23700,EPSG:4258,,hu_bme_geod,www.geod.bme.hu,Creative Commons Attribution 4.0 https://creativecommons.org/licenses/by/4.0/
geoid_eht2014.gtx,VERTICAL_OFFSET_GEOGRAPHIC_TO_VERTICAL,EPSG:4937,EPSG:8357,,,hu_bme_geod,www.geod.bme.hu,Creative Commons Attribution 4.0 https://creativecommons.org/licenses/by/4.0/

@rouault
Copy link
Member

rouault commented Jan 12, 2020

Can you address the review comments and rebase on top of latest master to fix the conflict in filelist.csv ?

@rouault
Copy link
Member

rouault commented Jan 29, 2020

WARNING: This repository is no longer open for adding new datasets. Starting with PROJ 7, the PROJ-data repository is the new home for PROJ datasets.

Go to its CONTRIBUTING instructions

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants