-
Notifications
You must be signed in to change notification settings - Fork 783
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
Update ESRI extras #904
Comments
So what do you propose exactly? That we update |
Hi Kristian, |
No. It's not like we do this every other day. Ideally the Esri guys would start by updating their file to include the |
OK thanks. When I look at the first entry in proj.4 |
No, I don't. It was added long before I started working on PROJ. You might get better answers on the mailing list. Note that the esri file hasn't been updated in ages. |
From what I understand, the esri file contains a lot of duplicate entries (2484 to be exact) with the standard EPSG database. But as only EPSG file has been maintained in the last 10 years, some of these duplicate entries have diverged. We can assume the correct ones are those maintained in the EPSG file. The ESRI file had a long-standing licensing issue described here. Would you accept such a PR? |
@rouault You probably know more about this stuff than I. Are there any problems in removing
Granted @rouault doesn't have a problem with this, yes. And especially if you add the |
I'm not clear how the nad/esri and nad/esri.extra files have been generated. Perhaps by running some modified scripts of GDAL's epsg_tr.py |
Good point. And there is a lot of different grids mentioned, several which are not covered by our datumgrid packages. Perhaps it is better remove the Esri files here and instead include the updated It would also be a nice step in the right direction with regards to separating resource files from the code. |
We have removed |
Hello,
We reuse the esri extra projection codes from proj.4 and found out D48_Slovenia_TM (102060) is missing.
It is defined in GDAL:
https://github.com/OSGeo/gdal/blob/master/gdal/data/esri_extra.wkt
And comes from this repository:
https://github.com/Esri/projection-engine-db-doc/blob/master/proj4/esri
The text was updated successfully, but these errors were encountered: