-
Notifications
You must be signed in to change notification settings - Fork 9
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
Proposal for geometry.uuid property #32
Comments
I'm concerned that this potentially violates §3.1.8 of RFC 7946:
More generally, rather than coming up with ad hoc ways of relating features, types, and geometries via UUID references, these kinds of relations should be expressed in RDF. If you want to assert something about a geometry, give it an |
Thanks, Ryan. Yes, Regarding RFC 7946, I don't think this proposal interferes with the semantics of the As for geometry objects, LP has already added And to clarify for other readers what RFC 7946 means:
Contrast with:
Do we understand the full implications? What could possibly go wrong?!? |
My concern here was that it changes the meaning of a GeometryCollection from a single composition of coordinates, to a set of related geometries grouped together under some heading… arguably having different geometries at different points in time already does that, but it that case it still makes sense to compose them. How would someone decide between using a |
I think that would be determined by the nature of the items in the dataset: this proposal has arisen only because we are creating sets of non-place objects (e.g. archaeological finds, drawings that can be geolocated) as noted in Issue #40. The data could be represented as a |
PROPOSAL: An optional
uuid
property that can be used in Geometry Collections to associate particular geometries with one or more of their parent feature's types. This might be used, for example, to indicate separate points for the creation, discovery, and repository of archaeological artefacts. Correspondingfeature
.types
would have ageometries
array of one or more such UUIDs.Outlined in LP.json.
The text was updated successfully, but these errors were encountered: