Hi Stefan,
Thanks for holding my feet to the fire on this issue. I've been traveling
the last week, but I'm ready to work on it again. I'll get back to you in
the next couple of days.
thanks,
Larry
On Thu, Apr 17, 2008 at 1:46 AM, Stefan Steiniger <[EMAIL PROTECTED]> wrote:
> Hei Larry,
>
> as I am reading the left over emails from last week:
> any news on this issue?
>
> stefan
>
> Larry Becker schrieb:
> > It looks like the code needs more work. I'm just using it as a quick
> > fix for now.
> >
> > thanks,
> > Larry
> >
> > On Fri, Apr 4, 2008 at 2:22 PM, Sunburned Surveyor
> > <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>
> wrote:
> >
> > You guys are definitely over my head, but it sounds like I should
> hold
> > off on commiting Larry's file. Is this correct?
> >
> > If Larry is still working on a fix I can wait until he tells me it
> is
> > ready to be commited.
> >
> > The Sunburned Surveyor
> >
> > On Fri, Apr 4, 2008 at 12:14 PM, Larry Becker
> > <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> wrote:
> > > Well, I have already found problems with this mod. I incorrectly
> > assumed
> > > that the clock-ness of the shells that were added as holes would
> > be fixed by
> > > later methods. That appears not to be true. Even though the mod
> > makes the
> > > holes appear correct, they still test bad with the QA tool. I'll
> > have to
> > > uncomment out the reverseRing loop.
> > >
> > > @Michaël, surely you didn't make the mistake of thinking I knew
> > what I was
> > > doing? ;-) My check for ((shells.size()>1) && (holes.size()==
> > 0)) is
> > > simply an attempt to minimize any damage my hack might do. Sorry
> > the code
> > > had so many changes. I knew I needed to subroutine the inline
> > code so I
> > > copied the work that was already done in geotools 2.5.
> > >
> > > regards,
> > > Larry
> > >
> > >
> > >
> > > On Fri, Apr 4, 2008 at 2:00 PM, Michaël Michaud
> > <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>
> > > wrote:
> > > > Martin Davis a écrit :
> > > >
> > > >
> > > >
> > > >
> > > > >Michaël Michaud wrote:
> > > > >
> > > > >
> > > > >>... about the second point, as you know SFS a no one, you may
> > have an
> > > > >>answer to the following question :
> > > > >>why polylines may be made of lines sharing their boundaries
> > > > >>and multipolygons cannot be made of polygons sharing their
> > edges (if
> > > > >>that's what "shells can touch at any finite number of points"
> > means)
> > > > >>
> > > > >>
> > > > >>
> > > > >That *is* what I meant by "shells can touch at any finite
> > number of
> > > points".
> > > > >
> > > > >I can't see into the minds of the standards authors, but
> > speaking as a
> > > > >developer who has implemented the standard, the restriction on
> how
> > > > >polygon shells touch makes for nicer topological assumptions,
> > and hence
> > > > >simpler code. With this rule you can assume that all edges in
> > polygon
> > > > >geometries are truly on the topological boundary - i.e that
> > they divide
> > > > >the polygon interior from the exterior. Without this
> > assumption, you
> > > > >have to basically do a "dissolve" to remove "interior" edges
> > before
> > > > >doing any further processing.
> > > > >
> > > > >Polylines don't offer quite such a strong topological
> > distinction (they
> > > > >don't divide the world into interior and exterior), so there
> > aren't any
> > > > >unpleasant implications to just letting their linework "go
> > anywhere it
> > > > >wants to".
> > > > >
> > > > >There are other assumptions which SFS could have made to make
> > processing
> > > > >simpler - such as enforcing an orientation for polygon rings.
> > But I
> > > > >guess they wanted to open the model up as much as reasonable,
> > so as not
> > > > >to make a whole bunch of geometry invalid. Although then they
> > could
> > > > >have allowed "inverted" polygons (self-touching at a point -
> > ArcSDE
> > > > >style) as well.
> > > > >
> > > > >M
> > > > >
> > > > >
> > > > All that makes sense to me.
> > > > Thanks for the detailed answer.
> > > >
> > > > Michaël
> > > >
> > > >
> > > >
> > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > >
> > > >
> > > >
> >
> -------------------------------------------------------------------------
> > > > Check out the new SourceForge.net Marketplace.
> > > > It's the best place to buy or sell services for
> > > > just about anything Open Source.
> > > >
> > >
> >
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> > > > _______________________________________________
> > > > Jump-pilot-devel mailing list
> > > > [email protected]
> > <mailto:[email protected]>
> > > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> > > >
> > >
> > >
> > >
> > > --
> > > http://amusingprogrammer.blogspot.com/
> > >
> >
> -------------------------------------------------------------------------
> > > Check out the new SourceForge.net Marketplace.
> > > It's the best place to buy or sell services for
> > > just about anything Open Source.
> > >
> >
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> > > _______________________________________________
> > > Jump-pilot-devel mailing list
> > > [email protected]
> > <mailto:[email protected]>
> > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> > >
> > >
> >
> >
> -------------------------------------------------------------------------
> > Check out the new SourceForge.net Marketplace.
> > It's the best place to buy or sell services for
> > just about anything Open Source.
> >
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> > _______________________________________________
> > Jump-pilot-devel mailing list
> > [email protected]
> > <mailto:[email protected]>
> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> >
> >
> >
> >
> > --
> > http://amusingprogrammer.blogspot.com/
> >
> >
> > ------------------------------------------------------------------------
> >
> >
> -------------------------------------------------------------------------
> > Check out the new SourceForge.net Marketplace.
> > It's the best place to buy or sell services for
> > just about anything Open Source.
> >
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> >
> >
> > ------------------------------------------------------------------------
> >
> > _______________________________________________
> > Jump-pilot-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
> Don't miss this year's exciting event. There's still time to save $100.
> Use priority code J8TL2D2.
>
> http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
> _______________________________________________
> Jump-pilot-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
--
http://amusingprogrammer.blogspot.com/
-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Jump-pilot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel