No, not yet. Will try and get on to this soon.
On Wed, Jul 29, 2015 at 5:43 AM, Kay, Jim <[email protected]> wrote: > Martin > > > > Did this get fixed in the trunk? > > Thanks. > > > > Jim > > > > *From:* Martin Davis [mailto:[email protected]] > *Sent:* 14 June 2015 22:06 > *To:* Kay, Jim > *Cc:* [email protected] > *Subject:* Re: [Jts-topo-suite-user] Geometry Union Fail with Polygon > createdfrom PackedCoordinateSequence > > > > Hmm, that's a bit of a shocker... amazing this bug hasn't surfaced before. > > > > The problem is that Translater implements CoordinateFilter, which is not > able to mutate non-Coordinate based CoordinateSequences. > > > > The fix may be simple. I'm thinking about creating a > MutatingCoordinateFilter which implements CoordinateSequenceFilter. With > any luck this can just be dropped in and will fix the problem. > > > > An alternative is to change the semantics of CoordinateFilter to force > mutations into the source geometry in a way which works with general > CoordinateSequences. This might be a bit slower, however, since it will > have to do a lot of checking to see if the coordinate should be updated, or > else update even if the Coordinate was not modified. Better I think to > make the ability to be mutated explicit. > > > > On Fri, Jun 12, 2015 at 7:07 AM, Kay, Jim <[email protected]> wrote: > > I have an incorrect from the union of two Polygons. > The first Polygon (g0) is created from a PackedCoordinateSequence. > The second Polygon (g1) is created from a CoordinateArray. > > The two Polygons are nearly on top of each other. > During the unioning process, 'OverlayOp.overlayOp' fails and so the code > defaults to 'SnapOverlayOp.overlayOp'. > In the subsequent code the removal of common bits using > 'removeCommonBits(Geometry geom)' fails to successfully apply the > translation to the geometry created from (g0) PackedCoordinateSequence. > Subsequently the code unions the unshifted g0 geometry with the shifted g1 > geometry, incorrectly producing a MultiPolygon. > > Java test code attached (with plotting; please maximise Window) > > I have copied some of the relevant method calls below. > >
------------------------------------------------------------------------------
_______________________________________________ Jts-topo-suite-user mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/jts-topo-suite-user
