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

Add NTv2 grids for Portugal, for convertion between Portuguese Lisbon Datum and Datum 73 to ETRS89 #54

Merged
merged 4 commits into from Sep 1, 2019

Conversation

PedroVenancio
Copy link
Contributor

No description provided.

@PedroVenancio
Copy link
Contributor Author

Hi @rouault

Can you please review my PR, with portuguese NTv2 grids?

These grids are already referenced in EPSG dataset:
https://github.com/OSGeo/proj.4/blob/master/data/sql/grid_transformation.sql#L193
https://github.com/OSGeo/PROJ/blob/master/data/sql/grid_transformation.sql#L194

Even so, it's necessary to update the grid_alternatives file?
https://github.com/OSGeo/PROJ/blob/master/data/sql/grid_alternatives.sql

Thank you very much!

@PedroVenancio
Copy link
Contributor Author

PedroVenancio commented Aug 29, 2019

I did the update to grid_alternatives.sql file

OSGeo/PROJ#1571

@rouault
Copy link
Member

rouault commented Aug 31, 2019

Note to other maintainers: wait for resolution of OSGeo/PROJ#1571 (comment) before merging this

@PedroVenancio
Copy link
Contributor Author

Note to other maintainers: wait for resolution of OSGeo/PROJ#1571 (comment) before merging this

I think it is ready now @rouault .

Thank you very much!

@kbevers kbevers added this to the proj-datumgrid-europe 1.5 milestone Sep 1, 2019
@kbevers kbevers merged commit 46e538f into OSGeo:master Sep 1, 2019
kbevers pushed a commit to OSGeo/PROJ that referenced this pull request Sep 1, 2019
grid_alternatives.sql: Add DLX_ETRS89_geo.gsb and D73_ETRS89_geo.gsb grids in grid_alternatives (added in proj-datumgrid-europe package per OSGeo/proj-datumgrid#54)
@PedroVenancio
Copy link
Contributor Author

Hi @kbevers @rouault

Do you have any idea about when proj-datumgrid-europe 1.5 will be released?

imagem

Thank you very much!

Best regards,
Pedro

@kbevers
Copy link
Member

kbevers commented Nov 22, 2019

It will be released alongside PROJ 6.3.0 January 1st (or thereabouts)

@PedroVenancio
Copy link
Contributor Author

Great!

Thanks @kbevers !

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.

None yet

3 participants