Hi Ticker, attached is the patch I've created with help of svn. It only reverts the changes made to MultiPolygonRelation in r3771.
I don't know yet why they cause trouble. Gerd ________________________________________ Von: mkgmap-dev <[email protected]> im Auftrag von Ticker Berkin <[email protected]> Gesendet: Freitag, 17. Februar 2017 10:32:11 An: [email protected] Betreff: Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch Hi Gerd Yes - if that fixes it. But in r3771 were there a couple of other changes not to MultiPolygon... haven't a lot more changes been made since to MultiPolygon... Ticker On Fri, 2017-02-17 at 09:19 +0000, Gerd Petermann wrote: > Hi Ticker, > > I think the problems were introduced with the highprec changes in > MultiPolygonRelation in r3771. > When I revert those the problem is gone. If you don't mind I commit > this revert. > > Gerd > ________________________________________ > Von: mkgmap-dev <[email protected]> im Auftrag > von Gerd Petermann <[email protected]> > Gesendet: Freitag, 17. Februar 2017 10:02:38 > An: Development list for mkgmap > Betreff: Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch > > Hi Ticker, > > don't need test data, found out what you meant. > > Gerd > ________________________________________ > Von: mkgmap-dev <[email protected]> im Auftrag > von Gerd Petermann <[email protected]> > Gesendet: Freitag, 17. Februar 2017 09:52:56 > An: Development list for mkgmap > Betreff: Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch > > Hi Ticker, > > okay, I'll merge the branch into trunk. > Please post some test data to show the coastline problem and I'll try > to fix it. > > Gerd > ________________________________________ > Von: mkgmap-dev <[email protected]> im Auftrag > von Ticker Berkin <[email protected]> > Gesendet: Freitag, 17. Februar 2017 09:50:17 > An: [email protected] > Betreff: Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch > > Hi Gerd > > I fixed this problem with r3807 in the split-shape branch and this > should probably be merged into trunk. > > Mike was also getting another problem that might be related to > generate-sea:extend-sea-sectors > > I don't know much about this area, but just having this option and > nothing else of significance, it seems as if the overview map has all > sea if it has a coastline > > Ticker > > On Thu, 2017-02-16 at 20:12 +0000, Gerd Petermann wrote: > > Hi Ticker, > > > > In case you need test data: I've uploaded a file : > > http://files.mkgmap.org.uk/download/334/88009211.osm.pbf > > Produces the error with r3811 without any options, just > > java -jar mkpgmap.jar 88009211.osm.pbf > > Seems to work okay with r3807. > > > > Gerd > > > > ________________________________________ > > Von: mkgmap-dev <[email protected]> im Auftrag > > von Mike Baggaley <[email protected]> > > Gesendet: Donnerstag, 16. Februar 2017 10:40:55 > > An: [email protected] > > Betreff: Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch > > > > HI Ticker, I'll package up some data for you. I have generate > > -sea:extend-sea-sectors in my options file, I think it will be > > related to that. > > > > Regards, > > Mike > > > > -----Original Message----- > > From: Ticker Berkin [mailto:[email protected]] > > Sent: 16 February 2017 08:17 > > To: [email protected] > > Subject: Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch > > > > Hi Mike > > > > I'll fix/improve the message. > > > > Can you tell if it is the sea polygons, the line 'coastline', or > > the > > island area (if your style generates this) that is corrupt. > > > > Maybe send your splitter options / areas.list, mkgmap options and > > style > > and I'll see if I can reproduce. > > > > Ticker > > > > On Thu, 2017-02-16 at 01:23 +0000, Mike Baggaley wrote: > > > HI Ticker, I've tried the patch without the --order-by-decreasing > > > -area option and it does now run without crashing. However, it > > > suffers from the same problem as I mentioned earlier when trying > > > - > > > -order-by-decreasing-area without the patch, in that the > > > coastline > > > is > > > corrupted. I haven't yet turned on detailed logging - I can do > > > that > > > if it would help. > > > > > > I note that in MapSplitter.java there is a new line: > > > > > > log.info("Single item larger that WANTED_MAX_AREA_SIZE", > > > area.getBounds().getCenter().toOSMURL()); > > > > > > I think this should say 'than' rather than 'that' and probably > > > WANTED_MAX_AREA_SIZE should be outside the string so we see an > > > actual > > > number. > > > > > > Regards, > > > Mike > > > > > > > > _______________________________________________ > > 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 > _______________________________________________ > 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 > _______________________________________________ > 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 _______________________________________________ mkgmap-dev mailing list [email protected] http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
x.patch
Description: x.patch
_______________________________________________ mkgmap-dev mailing list [email protected] http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
