[GEOS] #924: geometries returned by getGeometryN() should have the same SRID as collection

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[GEOS] #924: geometries returned by getGeometryN() should have the same SRID as collection

geos-2
#924: geometries returned by getGeometryN() should have the same SRID as
collection
-------------------------+--------------------------
 Reporter:  sirsigurd    |      Owner:  geos-devel@…
     Type:  enhancement  |     Status:  new
 Priority:  major        |  Milestone:
Component:  Default      |    Version:  master
 Severity:  Unassigned   |   Keywords:
-------------------------+--------------------------
 Currently geometries returned by getGeometryN() always have SRID=0,
 because SRID is dropped for elements during geometry collection
 construction since r3840. I think it would be more correct if collection
 components have the same SRID as collection.

--
Ticket URL: <https://trac.osgeo.org/geos/ticket/924>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).

_______________________________________________
geos-devel mailing list
[hidden email]
https://lists.osgeo.org/mailman/listinfo/geos-devel
Reply | Threaded
Open this post in threaded view
|

Re: [GEOS] #924: geometries returned by getGeometryN() should have the same SRID as collection

geos-2
#924: geometries returned by getGeometryN() should have the same SRID as
collection
-------------------------+---------------------------
 Reporter:  sirsigurd    |       Owner:  geos-devel@…
     Type:  enhancement  |      Status:  new
 Priority:  major        |   Milestone:
Component:  Default      |     Version:  master
 Severity:  Unassigned   |  Resolution:
 Keywords:               |
-------------------------+---------------------------
Changes (by sirsigurd):

 * cc: sirsigurd (added)


Comment:

 [https://github.com/libgeos/geos/pull/120 PR]

--
Ticket URL: <https://trac.osgeo.org/geos/ticket/924#comment:1>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).

_______________________________________________
geos-devel mailing list
[hidden email]
https://lists.osgeo.org/mailman/listinfo/geos-devel
Reply | Threaded
Open this post in threaded view
|

Re: [GEOS] #924: geometries returned by getGeometryN() should have the same SRID as collection

geos-2
In reply to this post by geos-2
#924: geometries returned by getGeometryN() should have the same SRID as
collection
-------------------------+---------------------------
 Reporter:  sirsigurd    |       Owner:  geos-devel@…
     Type:  enhancement  |      Status:  new
 Priority:  major        |   Milestone:
Component:  Default      |     Version:  master
 Severity:  Unassigned   |  Resolution:
 Keywords:               |
-------------------------+---------------------------

Comment (by dbaston):

 The proposed behavior (components have same SRID as collection) is
 consistent w/JTS:


 {{{
 -> GeometryFactory gf = new GeometryFactory(new PrecisionModel(), 4326);
 |  Added variable gf of type GeometryFactory with initial value
 org.locationtech.jts.geom.GeometryFactory@21213b92

 -> WKTReader reader = new WKTReader(gf);
 |  Added variable reader of type WKTReader with initial value
 org.locationtech.jts.io.WKTReader@3327bd23

 -> Geometry g = reader.read("GEOMETRYCOLLECTION(POINT (3 7), LINESTRING (0
 0, 1 1))");
 |  Added variable g of type Geometry with initial value GEOMETRYCOLLECTION
 (POINT (3 7), LINESTRING (0 0, 1 1))

 -> g.getSRID();
 |  Expression value is: 4326
 |    assigned to temporary variable $12 of type int

 -> g.getGeometryN(0).getSRID();
 |  Expression value is: 4326
 |    assigned to temporary variable $13 of type int

 -> g.getGeometryN(1).getSRID();
 |  Expression value is: 4326
 |    assigned to temporary variable $14 of type int

 }}}

 {{{

 }}}

--
Ticket URL: <https://trac.osgeo.org/geos/ticket/924#comment:2>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).

_______________________________________________
geos-devel mailing list
[hidden email]
https://lists.osgeo.org/mailman/listinfo/geos-devel
Reply | Threaded
Open this post in threaded view
|

Re: [GEOS] #924: geometries returned by getGeometryN() should have the same SRID as collection

geos-2
In reply to this post by geos-2
#924: geometries returned by getGeometryN() should have the same SRID as
collection
-------------------------+---------------------------
 Reporter:  sirsigurd    |       Owner:  geos-devel@…
     Type:  enhancement  |      Status:  closed
 Priority:  major        |   Milestone:
Component:  Default      |     Version:  master
 Severity:  Unassigned   |  Resolution:  fixed
 Keywords:               |
-------------------------+---------------------------
Changes (by Sergey Fedoseev <fedoseev.sergey@…>):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"33c1e7bc78dbfffcaf9a05744e49b76d91d21d2f/git" 33c1e7b/git]:
 {{{
 #!CommitTicketReference repository="git"
 revision="33c1e7bc78dbfffcaf9a05744e49b76d91d21d2f"
 Fix #924: Geometries returned by getGeometryN() now have the same SRID as
 collection.
 }}}

--
Ticket URL: <https://trac.osgeo.org/geos/ticket/924#comment:3>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).

_______________________________________________
geos-devel mailing list
[hidden email]
https://lists.osgeo.org/mailman/listinfo/geos-devel