Re: [geos-devel] [GEOS] #985: Replace sort to min_element in SubgraphDepthLocater::getDepth

2019-09-09 Thread GEOS
#985: Replace sort to min_element in SubgraphDepthLocater::getDepth
--+--
 Reporter:  dkvash|   Owner:  dbaston
 Type:  patch |  Status:  new
 Priority:  trivial   |   Milestone:
Component:  Default   | Version:  master
 Severity:  Idea  |  Resolution:
 Keywords:  SubgraphDepthLocater  |
--+--
Changes (by dbaston):

 * owner:  geos-devel@… => dbaston


-- 
Ticket URL: 
GEOS 
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite 
(JTS).
___
geos-devel mailing list
geos-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geos-devel

Re: [geos-devel] [GEOS] #985: Replace sort to min_element in SubgraphDepthLocater::getDepth

2019-09-09 Thread GEOS
#985: Replace sort to min_element in SubgraphDepthLocater::getDepth
--+---
 Reporter:  dkvash|   Owner:  geos-devel@…
 Type:  patch |  Status:  new
 Priority:  trivial   |   Milestone:
Component:  Default   | Version:  master
 Severity:  Idea  |  Resolution:
 Keywords:  SubgraphDepthLocater  |
--+---
Changes (by dkvash):

 * priority:  major => trivial
 * severity:  Unassigned => Idea


-- 
Ticket URL: 
GEOS 
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite 
(JTS).
___
geos-devel mailing list
geos-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geos-devel

[geos-devel] [GEOS] #985: Replace sort to min_element in SubgraphDepthLocater::getDepth

2019-09-09 Thread GEOS
#985: Replace sort to min_element in SubgraphDepthLocater::getDepth
+--
 Reporter:  dkvash  |  Owner:  geos-devel@…
 Type:  patch   | Status:  new
 Priority:  major   |  Milestone:
Component:  Default |Version:  master
 Severity:  Unassigned  |   Keywords:  SubgraphDepthLocater
+--
 SubgraphDepthLocater::getDepth uses heavy std::sort to find minimum
 DepthSegment, this patch replaces sort to a more appropriate
 std::min_element.

 Origin implementation also relies on a Collections.min instead of heavy
 sort.
 
https://github.com/locationtech/jts/blob/jts-1.16.1/modules/core/src/main/java/org/locationtech/jts/operation/buffer/SubgraphDepthLocater.java#L53

-- 
Ticket URL: 
GEOS 
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite 
(JTS).
___
geos-devel mailing list
geos-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geos-devel

Re: [geos-devel] [GEOS] #983: Problem with 3.7 branch and shapely

2019-09-09 Thread GEOS
#983: Problem with 3.7 branch and shapely
+---
 Reporter:  aom |   Owner:  geos-devel@…
 Type:  defect  |  Status:  new
 Priority:  major   |   Milestone:  3.7.3
Component:  Default | Version:  3.7.0
 Severity:  Unassigned  |  Resolution:
 Keywords:  |
+---

Comment (by sgillies):

 Here's a workaround for now: run your python program with -O to remove
 that assertion. See
 https://docs.python.org/3/using/cmdline.html#cmdoption-o.

-- 
Ticket URL: 
GEOS 
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite 
(JTS).
___
geos-devel mailing list
geos-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geos-devel

Re: [geos-devel] [GEOS] #983: Problem with 3.7 branch and shapely

2019-09-09 Thread GEOS
#983: Problem with 3.7 branch and shapely
+---
 Reporter:  aom |   Owner:  geos-devel@…
 Type:  defect  |  Status:  new
 Priority:  major   |   Milestone:  3.7.3
Component:  Default | Version:  3.7.0
 Severity:  Unassigned  |  Resolution:
 Keywords:  |
+---

Comment (by sgillies):

 I've changed Shapely to make it more tolerant. But you'll have to use
 Shapely's master branch. The 1.7 release is a ways out yet.

-- 
Ticket URL: 
GEOS 
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite 
(JTS).
___
geos-devel mailing list
geos-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geos-devel