Hi Martin,
Well, the fix turned out to be spectacularly easy after all. The sort was only used to find the minimum DepthSegment, so I switched to using Collections.min(), which doesn't have such strict requirements on the ordering.

https://sourceforge.net/p/jts-topo-suite/code/959/
https://sourceforge.net/p/jts-topo-suite/code/960/

The DepthSegment compareTo still does not obey the contract, unfortunately. It's difficult to come up with a consistent linear ordering for 2D segments.

It would be great if someone can build this and give it a test with real data.
What is this ordering made for, and what do you mean by linear ordering ? Isn't LineSegment#compareTo method enough to get a strict ordering over segments for example ?

Michaël

On Wed, Feb 4, 2015 at 7:31 PM, Martin Davis <[email protected] <mailto:[email protected]>> wrote:

    Some progress to report on this issue...   I now have a test
    harness that will cause the failure. I've made some progress
    towards implementing the fix, but it's turning out to be quite
    complex to handle all cases. Hopefully I'll be able to resolve
    this completely soon..

    On Wed, Feb 4, 2015 at 11:00 AM, Martin Davis <[email protected]
    <mailto:[email protected]>> wrote:

        Well, it would be great to get this fixed.  But I have no test
        case that I can reproduce the issue with.  I need this to
        determine what the fix should be (the code logic is a bit
        complex).

        Ideally this would be in the form of some synthetic
        DepthSegments which when sorted produce the issue.  Failing
        that, a geometry and the call that produces the error...

        On Wed, Feb 4, 2015 at 7:00 AM, Marcell Roth
        <[email protected] <mailto:[email protected]>> wrote:

            Hi,
            we still have issues with Java 7 and the problem described
            in the following discussion:
            _http://sourceforge.net/p/jts-topo-suite/mailman/message/30364799/_
            Our Java version:
            java version "1.7.0_55"
            Java(TM) SE Runtime Environment (build 1.7.0_55-b13)
            Java HotSpot(TM) 64-Bit Server VM (build 24.55-b03, mixed
            mode)
            Will the next release of JTS support Java 7 or even Java
            8? Do you know approximately when the next version will be
            released?
            Kind regards,
            Marcell
            *Marcell Roth*
            Software Engineer
            SDI & E-Government
            *con terra *- Gesellschaft für Angewandte
            Informationstechnologie mbH
            Martin-Luther-King-Weg 24
            48155 Münster
            Telefon +49 89 207 005 2444
            <tel:%2B49%2089%20207%20005%202444>
            [email protected]_ <mailto:[email protected]>
            _www.conterra.de_ <http://www.conterra.de/>
            Geschäftsführung: Karl Wiesmann, Uwe König
            Amtsgericht Münster HRB 4149
            Ust.IDNr. DE 162264061

            
------------------------------------------------------------------------------
            Dive into the World of Parallel Programming. The Go
            Parallel Website,
            sponsored by Intel and developed in partnership with
            Slashdot Media, is your
            hub for all things parallel software development, from
            weekly thought
            leadership blogs to news, videos, case studies, tutorials
            and more. Take a
            look and join the conversation now.
            http://goparallel.sourceforge.net/
            _______________________________________________
            Jts-topo-suite-user mailing list
            [email protected]
            <mailto:[email protected]>
            https://lists.sourceforge.net/lists/listinfo/jts-topo-suite-user






------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/


_______________________________________________
Jts-topo-suite-user mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jts-topo-suite-user

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Jts-topo-suite-user mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jts-topo-suite-user

Reply via email to