[mkgmap-dev] RE Commit r3801: merge split-shape branch

Fri Feb 17 09:32:11 GMT 2017

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 <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> von Gerd Petermann <GPetermann_muenchen at hotmail.com>
> 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 <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> von Gerd Petermann <GPetermann_muenchen at hotmail.com>
> 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 <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> von Ticker Berkin <rwb-mkgmap at jagit.co.uk>
> Gesendet: Freitag, 17. Februar 2017 09:50:17
> An: mkgmap-dev at lists.mkgmap.org.uk
> 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 <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> > von Mike Baggaley <mike at tvage.co.uk>
> > Gesendet: Donnerstag, 16. Februar 2017 10:40:55
> > An: mkgmap-dev at lists.mkgmap.org.uk
> > 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:rwb-mkgmap at jagit.co.uk]
> > Sent: 16 February 2017 08:17
> > To: mkgmap-dev at lists.mkgmap.org.uk
> > 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
> > mkgmap-dev at lists.mkgmap.org.uk
> > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
> > _______________________________________________
> > mkgmap-dev mailing list
> > mkgmap-dev at lists.mkgmap.org.uk
> > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev