-
Notifications
You must be signed in to change notification settings - Fork 13
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
Using GeometryCollection as a feature's place is underspecified #71
Comments
Good points @IvanSanchez. Maybe we should exclude On the last comment:
Currently they cannot. |
My personal stance is that any restrictions to "2D geometries" should scope into e.g. a restriction on how a
I'll add: my stance is that (of course this should only apply to the 3D conformance class) |
I think that we must not support mixed CRS in GeometryCollection. They are not allowed in the OpenGIS® Implementation Standard for Geographic information - Simple feature access - Part 1: Common architecture nor in Open GIS Consortium, Inc.
The GeoJSON RFC https://www.rfc-editor.org/rfc/rfc7946 defines "GeoJSON comprises the seven concrete geometry types defined in the OpenGIS Simple Features Implementation Specification for SQL". It does not define that when it comes to GeometryCollection then GeoJSON is more flexible than OGC. |
That's a very convincing argument! I now fully agree that Can we please amend the JSON-FG spec so that it mentions that bit of the OGC SFA? That should clear things up. |
I agree, good point. In addition, the OpenGIS Simple Features Implementation Specification is already a normative reference in the spec. |
Meeting 2022-10-31:
|
Meeting 2023-05-29: The first change is included in #89, but we want to keep the discussion about the need for a GeometryCollection that supports all JSON-FG geometries, too. |
This is regarding sections 7.3 and 8.3 of the draft spec: the "Geometry" section of the FG building blocks and the Core Requirements class, respectively.
My current understanding is that I can use a
GeometryCollection
as theplace
property of a feature - nothing in the draft spec prevents me from doing so. I assume that, since JSON-FG must cover all use cases of GeoJSON, havingGeometryCollection
s is an expected use case.However, while there are extra restrictions on using GeoJSON geometries like
Point
,MultiPoint
,LineString
,MultiLineString
,Polygon
orMultiPolygon
, there are no such restrictions onGeometryCollection
s. See 8.3.3 and 8.3.4.GeometryCollection
s pose a challenge to the scoping rules ofcoordRefSys
. Consider the following example, which joins together the centroid of the Madrid-Barajas airport with two of its runways:My current scoping scoping rules for
coordRefSys
say:My understanding is that, since every
Geometry
inside aGeometryCollection
is... well, a geometry, then the scoping rule applies, and the above example makes sense.My own client implementation of JSON-FG can easily deal with this case, but other client implementations might not have it so easy due to architectural constrains. In any case, I feel there's a need for clarification on how
coordRefSys
inGeometryCollections
should work.It s also unclear whether 3D geometries (
Polyhedron
, etc) can be part of aGeometryCollection
.The text was updated successfully, but these errors were encountered: