Hi Thorsten, please try the attached patch. It is not elegant, but I think it will solve the problem. As mentioned before, the problem is caused by rounding errors, so I've coded a check that detects when this situation happens. Area parts that cause this error are removed. If you like, you can unncomment the lines around GpxCreator to write the deleted parts to files.
ciao, Gerd > Date: Wed, 25 Jan 2012 08:11:30 +0100 > From: [email protected] > To: [email protected] > Subject: Re: [mkgmap-dev] mkgmap r2179: Still NullPointerException in > BoundaryUtil > > > > Hi Gerd, > > On Tue, Jan 24, Gerd Petermann wrote: > > > > > Hi Thorsten, > > > > don't worry, the patch was not meant to fix your problem. > > It just adds a method to class Way, but the method is not even called > > anywhere. > > Sorry, was clearly too late yesterday for me, I should have seen that > the new method isn't called anywhere :( > > Thorsten > > > > > > Date: Tue, 24 Jan 2012 22:57:19 +0100 > > > From: [email protected] > > > To: [email protected] > > > Subject: Re: [mkgmap-dev] mkgmap r2179: Still NullPointerException > > > in BoundaryUtil > > > > > > > > > Hi Gerd, > > > > > > On Tue, Jan 24, Gerd Petermann wrote: > > > > > > > Hi WanMil, > > > > > > > > I have coded a small test that tries to detect these "spike-like" > > > > areas. Idea: > > > > If the area size is much smaller (I used 20 as a threshold) than that > > > > of the bounding box we should remove > > > > the way from the area. > > > > > > This patch does at least not fix my problem, still get exact the > > > same stack trace. > > > > > > Thorsten > > > > > > > > > > I am not sure regarding the threshold value, but I think the principle > > > > is ok. > > > > What do you think? > > > > > > > > Gerd > > > > > > > > > > > > > Date: Tue, 24 Jan 2012 08:04:47 -0800 > > > > > From: [email protected] > > > > > To: [email protected] > > > > > Subject: Re: [mkgmap-dev] mkgmap r2179: Still NullPointerException in > > > > > BoundaryUtil > > > > > > > > > > Hi Thorsten, > > > > > > > > > > I can confirm this problem. I can repdroduce it with a "boundary" > > > > > which > > > > > contains these 4 points: > > > > > [2412980/371559, 2412981/371558, 2412992/371544, 2412980/371559] > > > > > The Way.clockwise() method says it is counter-clockwise. > > > > > Besides rounding errors, it is a line, not a shape, and I guess that > > > > > causes > > > > > the problem. The clockwise() method calculates an area size of 3, > > > > > which is > > > > > almost zero. > > > > > > > > > > It is quite likely that the intersection of two areas produces these > > > > > "nearly > > > > > empty" (parts of) areas, what we need is a filter that throws them > > > > > away. > > > > > > > > > > I am not sure if WanMil is already working on this? > > > > > > > > > > Gerd > > > > > > > > > > > > > > > Thorsten Kukuk wrote > > > > > > > > > > > > Hi, > > > > > > > > > > > > I compiled mkgmap r2179 and tried to create the bounds files with > > > > > > it: > > > > > > > > > > > > osmconvert data/planet/planet-120122.osm.pbf --out-o5m > > > > > > osmfilter data/boundaries/world-20120122.o5m --keep-nodes= > > > > > > '--keep-ways-relations=boundary=administrative =postal_code > > > > > > postal_code=' > > > > > > mkgmap --createboundsfile=data/boundaries/boundaries-20120122.osm.gz > > > > > > --bounds=data/boundaries/20120122 > > > > > > Schwerwiegend (BoundarySaver): Calculate bbox to > > > > > > (-90.12222290039062,-180.24444580078125) to > > > > > > (82.6120376586914,179.17156219482422) > > > > > > Exception in thread "main" java.lang.NullPointerException > > > > > > at > > > > > > uk.me.parabola.mkgmap.reader.osm.boundary.BoundaryUtil.loadBoundaryFile(BoundaryUtil.java:169) > > > > > > at > > > > > > uk.me.parabola.mkgmap.reader.osm.boundary.BoundaryPreparer.workoutBoundaryRelations(BoundaryPreparer.java:144) > > > > > > at > > > > > > uk.me.parabola.mkgmap.reader.osm.boundary.BoundaryPreparer.run(BoundaryPreparer.java:110) > > > > > > at uk.me.parabola.mkgmap.main.Main.endOptions(Main.java:333) > > > > > > at > > > > > > uk.me.parabola.mkgmap.CommandArgsReader.readArgs(CommandArgsReader.java:126) > > > > > > > > > > > > at uk.me.parabola.mkgmap.main.Main.main(Main.java:112) > > > > > > Command exited with non-zero status 1 > > > > > > > > > > > > > > > > > > Thorsten > > > > > > > > > > > > -- > > > > > > Thorsten Kukuk, Project Manager/Release Manager SLES > > > > > > SUSE LINUX Products GmbH, Maxfeldstr. 5, D-90409 Nuernberg > > > > > > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG > > > > > > Nürnberg) > > > > > > _______________________________________________ > > > > > > mkgmap-dev mailing list > > > > > > [email protected] > > > > > > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > > > > > > > > > > > > > > > > > > > > -- > > > > > View this message in context: > > > > > http://gis.638310.n2.nabble.com/mkgmap-r2179-Still-NullPointerException-in-BoundaryUtil-tp7215004p7220841.html > > > > > Sent from the Mkgmap Development mailing list archive at Nabble.com. > > > > > _______________________________________________ > > > > > mkgmap-dev mailing list > > > > > [email protected] > > > > > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > > > > > > > > > > > > > > _______________________________________________ > > > > mkgmap-dev mailing list > > > > [email protected] > > > > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > > > > > -- > > > Thorsten Kukuk, Project Manager/Release Manager SLES > > > SUSE LINUX Products GmbH, Maxfeldstr. 5, D-90409 Nuernberg > > > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg) > > > _______________________________________________ > > > mkgmap-dev mailing list > > > [email protected] > > > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > > > _______________________________________________ > > mkgmap-dev mailing list > > [email protected] > > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > -- > Thorsten Kukuk, Project Manager/Release Manager SLES > SUSE LINUX Products GmbH, Maxfeldstr. 5, D-90409 Nuernberg > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg) > _______________________________________________ > mkgmap-dev mailing list > [email protected] > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
verify_boundary.patch
Description: Binary data
_______________________________________________ mkgmap-dev mailing list [email protected] http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
