-
-
Notifications
You must be signed in to change notification settings - Fork 591
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
CRS updates #725
CRS updates #725
Conversation
See #713
Heads-up @Nowosad the intention with this PR is to complete Chapter and therefore (I think) most of the stuff needed to submit Part 1 for review. Sorry it's taken so long. Key points: we should not mention 'WKT2' as it's confusing, WKT2 is and in the future will be synonymous with WKT representation of CRSs. I have read and linked to key related docs on CRSs. One thing that surprised me: it does not seem that the AUTHORITY:CODE string convention in https://docs.opengeospatial.org/is/18-010r7/18-010r7.html#37 now given the proper citation it deserves, the most important doc here The QGIS docs which show 5 CRS identifier types: https://docs.qgis.org/3.16/en/docs/pyqgis_developer_cookbook/crs.html?highlight=srid PostGIS docs which is where I think the term 'SRID' comes from. I think 'identifier' is better for our purposes: https://postgis.net/workshops/postgis-intro/projection.html As others have said this is a minefield. But I'm confident the updated text is accurate and better. Needed to rework these earlier sections before getting to 7.4. Any comments v. welcome. |
Hi @Robinlovelace,
J |
Hi @Nowosad all good questions and it's a minefield so worth discussing. Responses soon... |
Regarding your point 1:
The fact that the WKT representation has changed over time is not relevant for most users and will be come less relevant, tending towards obsolete, in the coming years. Good idea to mention it. I think a footnote in C7 would be the best place for that, e.g. here: That's my default plan, it fits well there because that footnote also mentions the increasingly obsolete |
Good idea to focus on what PROJ and GDAL accept, I was looking (perhaps too much) at the PostGIS docs. Yes of course the identifier is only a look-up and that the CRS is stored in the WKT string. That is now more clear I think. If we had links showing that PROJ/GDAL accept the |
Good point on that, I think clarifying that and adding a cross reference to the subsequent section is a good idea. However, I think the QGIS example shows that there are alternative CRS identifiers other than the 'SRID'. |
Yes according to PostGIS documentation. But where else is that term used? I'm not sure that in PostGIS docs they would say that I think if we define CRS identifiers early and clearly, with reference to the OGC standard which calls them identifiers, it will be clear to readers. Pros and cons. ATM I'm thinking the advantages of just 'identifier' used by OGC outweigh the advantages of |
Another observation: there is really good documentation here which mentions identifiers but avoids the acronym SRID: https://proj.org/operations/operations_computation.html However there is mention of SRIDs here: https://proj.org/faq.html |
But that seems to be the only mention of the term: https://www.google.com/search?q=srid+%22spatial+reference%22+site%3Aproj.org |
In GDAL docs there seems to be no mention or srids: https://www.google.com/search?q=srid+%22spatial+reference%22+site%3Agdal |
See #713