-
Notifications
You must be signed in to change notification settings - Fork 18
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
Reword specification to be consistent on the mandatory nature of CRS VLR #85
Comments
I don't know when that VLR got made required. I don't think it was required in older versions of the document, and indeed it is legitimate to have data without CRS. IMO, the CRS statements in Section 3 should be relaxed. |
@hobu Why would we relax this requirement? Although nothing is necessarily broken by excluding the CRS information, having that requirement in the LAS specification has in the past given agencies leverage to require their data providers to put in the effort to add encoding, rather than relying on outside definitions. The only time this has broken down for me is when people need a local CRS that has no projection definition, in which case we made an exception to this rule. I don't know the history well enough to know when the CRS VLR became required, but I think @pchilds makes sense to explicitly call out the CRS VLR as being required. It never occurred to me that someone might skip over the CRS section. |
|
Through the las specifications 1.0 to 1.3 there has been the statements: "The projection information for the point data is required...", "There is one mandatory Variable Length Record, GeoKeyDirectoryTag...", "Only the GeoKeyDirectoryTag record is required". Afaik the CRS has always been mandatory, it is only with v1.4 as to whether to place it in a VLR or EVLR and as a result the statements as to its requirements have been shunted further down in the document. |
My experience with PDAL development has been to somewhat commonly see data without any VLRs, especially data in scanner-local coordinate systems as part of a larger project. We shouldn't change the specification or relax the requirement given the history @esilvia, but do recognize there is a lot of data potentially out there that doesn't follow along. |
I find it is rare to come across a las file that even has the mandatory VLR for the coordinate system. It does not help that the specification is inconsistent in this regard (giving the strong impression in reading from the beginning that VLRs are only ever optional):
I believe many software developers once obtaining this impression will skip reading beyond the core of the specification to discover that the use of a CRS VLR is actually required by the specification.
The text was updated successfully, but these errors were encountered: