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

Add Polish geoid model PL-geoid2021 #105

Merged
merged 2 commits into from
Jul 11, 2023

Conversation

AlexBass05
Copy link
Contributor

@AlexBass05 AlexBass05 commented Jul 6, 2023

EPSG v10.091 added the missing transformation for the latest Polish geoid model PL-geoid2021:

This unblocks the addition of that geoid model to PROJ.

License and source file: http://www.gugik.gov.pl/bip/prawo/modele-danych

This PR depends on OSGeo/PROJ#3812.

Related PR in PROJ: OSGeo/PROJ#3817

@rouault
Copy link
Member

rouault commented Jul 9, 2023

@AlexBass05
can you modify (and re-run) the build script to include a gdalwarp stage like the one used for pl_gugik_geoid2011-PL-EVRF2007-NH.tif, so that the raster produced is "north-up", which is a more usual practice

@AlexBass05 AlexBass05 force-pushed the add-polish-geoid2021 branch from 629d687 to 634768f Compare July 10, 2023 08:18
@AlexBass05
Copy link
Contributor Author

AlexBass05 commented Jul 10, 2023

@AlexBass05 can you modify (and re-run) the build script to include a gdalwarp stage like the one used for pl_gugik_geoid2011-PL-EVRF2007-NH.tif, so that the raster produced is "north-up", which is a more usual practice

@rouault
Thanks for pointing this out. Done. The respective warning issued by check_gtiff_grid.py is gone now.
I regenerated the index again.

@rouault rouault merged commit 49761e6 into OSGeo:master Jul 11, 2023
@rouault rouault added this to the 1.15 milestone Jul 11, 2023
@AlexBass05 AlexBass05 deleted the add-polish-geoid2021 branch April 2, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants