Larry, You wrote: " I was waiting for one of the other developers experienced with the shapfile reader to comment on the change."
I don't think we have one. :] If there are not objections I will commit the change today when I commit Peppe's I18N file changes. Thank you for your work. I appreciate you feeding your improvements from SkyJUMP back into OpenJUMP. The Sunburned Surveyor On Fri, Apr 4, 2008 at 8:12 AM, Larry Becker <[EMAIL PROTECTED]> wrote: > Hi Sunburned, > > I believe that SkyJUMP and OpenJUMP are the same for shapefile code. It > should only be necessary to copy the file over to the OpenJUMP project. I > was waiting for one of the other developers experienced with the shapfile > reader to comment on the change. > > regards, > Larry > > > > On Fri, Apr 4, 2008 at 9:58 AM, Sunburned Surveyor > <[EMAIL PROTECTED]> wrote: > > Larry, > > > > Will we need to make mods to OpenJUMP to benefit from your > > improvements? I am a little confused as to whether this is a fix you > > made to a tool specific to SkyJUMP, or if we need to fix OpenJUMP's > > Shapefile reader. > > > > The Sunburned Surveyor > > > > > > > > > > On Thu, Apr 3, 2008 at 2:11 PM, Larry Becker <[EMAIL PROTECTED]> > wrote: > > > Thanks for the quick answer, Martin. I've made a mod to PolygonHandler > in > > > SkyJUMP and committed my change. You can browse it at: > > > > > > > http://skyjump.cvs.sourceforge.net/skyjump/skyjump/org/geotools/shapefile/PolygonHandler.java?view=markup > > > > > > My mod begins in read() at "// quick optimization:". I used some of the > > > geotools 2.5 code to help with the update. In essence, I do a check for > > > ((shells.size()>1) && (holes.size()== 0)) before trying anything new to > > > minimize the impact of the change. Then I call the shellsOverlap() > method > > > which returns the index of the first shell that contains another. I > then > > > make the big assumption that all the rest are holes. I think I can get > away > > > with this because the assignHolesToShells() method converts any > unassigned > > > holes back to shells. > > > > > > I would appreciate any comments. > > > > > > regards, > > > Larry Becker > > > > > > > > > > > > > > > On Thu, Apr 3, 2008 at 12:44 PM, Martin Davis <[EMAIL PROTECTED]> > > > wrote: > > > > Ugh. > > > > > > > > Larry, I think your approach is probably what's necessary. I would > > > > suggest NOT trying to do a JTS contains() to implement the "wholly > > > > inside" test - it would be very slow (although the new > PreparedGeometry > > > > work would make it a lot faster - but it would still be slow). > > > > > > > > You might consider just check say 3 points of the ring to see if they > > > > are wholly inside - if so, assume it is a hole. That should be a lot > > > > faster. > > > > > > > > > > > > > > > > > > > > Larry Becker wrote: > > > > > I've found a shapefile that has what JTS thinks is a topology error > of > > > > > "overlapping shells". In ESRI ArcMap it displays correctly as a > shell > > > > > polygon with a hole, but in JUMP, it displays as overlapping > > > > > polygons. It fails the QA "Basic Topology" test. I have verified > > > > > that the "hole" polygon is not CCW (counter clockwise) and this is > > > > > being interpreted as a shell by the org.geotools.PolygonHandler. > > > > > > > > > > It looks like another case where ESRI isn't following their own > > > > > specifications. Any suggestions? I don't like to "fix" customer's > > > > > data when it works fine in their ESRI system. > > > > > > > > > > I'm considering modifying the PolygonHandler code to test all of the > > > > > polygons in a multipolygon shape to determine if they are completely > > > > > inside, and then reversing the point order to force CCW. This might > > > > > make shapefiles read slightly slower. > > > > > > > > > > regards, > > > > > > > > > > Larry Becker > > > > > > > > > > -- > > > > > 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 > > > > > Jump-pilot-devel@lists.sourceforge.net > > > > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > > > > > > > > > > > > -- > > > > Martin Davis > > > > Senior Technical Architect > > > > Refractions Research, Inc. > > > > (250) 383-3022 > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > 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 > > > > Jump-pilot-devel@lists.sourceforge.net > > > > 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 > > > Jump-pilot-devel@lists.sourceforge.net > > > 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 > > Jump-pilot-devel@lists.sourceforge.net > > 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 > Jump-pilot-devel@lists.sourceforge.net > 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 Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
