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

Fri Feb 17 08:52:56 GMT 2017

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