[mkgmap-dev] Commit r4470: genSea.patch by Ticker Berkin

Fri Mar 20 21:35:25 GMT 2020

Hi Ticker,

The image was contained more than a single tile - the tile I was referring to was a rectangle of which the white area was the bottom right corner. The second image shows the border between that tile (on the right) and the adjacent tile on the left. The original tile did in fact contain some sea correctly rendered in an inlet originating at Kingswear which is just on the edge of the white area. 

I used just generate-sea without any of the other options this time, but have used extend-sea-vectors and close-gaps without any obvious effect. I suspect that the problem with my outline is not that there are gaps, but that there may be loops where the coastline crosses itself.

Regards,
Mike

-----Original Message-----
From: Ticker Berkin [mailto:rwb-mkgmap at jagit.co.uk] 
Sent: 20 March 2020 09:45
To: Development list for mkgmap <mkgmap-dev at lists.mkgmap.org.uk>
Subject: Re: [mkgmap-dev] Commit r4470: genSea.patch by Ticker Berkin

Hi Mike

Another question:

Was the image attached to the first mail a single tile?

Ticker

On Fri, 2020-03-20 at 09:39 +0000, Ticker Berkin wrote:
> Hi Mike
> 
> Errors like are normally because the tile is at the edge of your
> downloaded area and includes incomplete coastline that doesn't reach
> the tile edge and/or has gaps where there are estuaries, harbours,
> etc.
> 
> If this is not the case then there are either errors in the
> natural=coastline data or in the logic of SeaGenerator.java
> 
> Depending on your --generate-sea options, the code tries to resolve
> this in different manners (sea-sectors, extend-sea-sectors, no-sea
> -sectors). None of these are guaranteed to work, but I tried to
> improve
> it so that it was more likely that sea was land-flooded rather than
> the
> other way around.
> 
> What are you generate-sea options. I've found that:
>  --generate-sea=multipolygon,extend-sea-sectors,close-gaps=500
> works well most of the time
> 
> Ticker
> 
> 
> On Fri, 2020-03-20 at 07:34 +0000, Mike Baggaley wrote:
> > Just noticed the log file has the following in it:
> > 
> >  66973072.o5m: Adjacent coastlines hit tile edge in same direction
> > 3.512082099914551 null
> >  66973072.o5m: Adjacent coastlines hit tile edge in same direction
> > 3.904301643371582 WAY: 4611686018492528634 null(50.39497375488281/
> > -3.603515625) []
> > 
> > This is the offending tile, although there are other tiles showing
> > the same message:
> > 
> >   66973028.o5m: Adjacent coastlines hit tile edge in same direction
> > 3.68414306640625 null
> >   66973028.o5m: Adjacent coastlines hit tile edge in same direction
> > 3.6845932006835938 WAY: 4611686018454067807
> > null(51.540770530700684/0.5712890625) []
> >   66973031.o5m: Adjacent coastlines hit tile edge in same direction
> > 1.3157111273871527 null
> >   66973031.o5m: Adjacent coastlines hit tile edge in same direction
> > 1.5220565795898438 WAY: 4611686018456515937
> > null(51.62250280380249/0.5712890625) []
> >   66973063.o5m: Adjacent coastlines hit tile edge in same direction
> > 0.964984130859375 null
> >   66973063.o5m: Adjacent coastlines hit tile edge in same direction
> > 0.968072509765625 WAY: 4611686018481807136 null(50.8447265625/
> > -1.112666130065918) []
> >   66973064.o5m: Adjacent coastlines hit tile edge in same direction
> > 2.0334129333496094 null
> >   66973064.o5m: Adjacent coastlines hit tile edge in same direction
> > 2.1257583618164064 WAY: 4611686018481855529 null(50.8447265625/
> > -1.1539077758789062) []
> >   66973071.o5m: Adjacent coastlines hit tile edge in same direction
> > 1.4875097274780273 null
> >   66973071.o5m: Adjacent coastlines hit tile edge in same direction
> > 1.493368148803711 WAY: 4611686018490569733 null(50.534770488739014/
> > -3.603515625) []
> >   66973216.o5m: Adjacent coastlines hit tile edge in same direction
> > 3.6122945149739585 null
> >   66973216.o5m: Adjacent coastlines hit tile edge in same direction
> > 3.6950721740722656 WAY: 4611686018642067673 null(54.13313627243042/
> > -2.8125) []
> >   66973233.o5m: Adjacent coastlines hit tile edge in same direction
> > 1.8426347620346966 null
> >   66973233.o5m: Adjacent coastlines hit tile edge in same direction
> > 1.9191019394818474 WAY: 4611686018657352840 null(54.21201467514038/
> > -2.8125) []
> > 
> > Do these messages indicate errors in the coastline data?
> > 
> > Regards,
> > Mike
> > 
> > -----Original Message-----
> > From: Mike Baggaley [mailto:mike at tvage.co.uk] 
> > Sent: 19 March 2020 23:44
> > To: mkgmap-dev at lists.mkgmap.org.uk
> > Subject: Re: [mkgmap-dev] Commit r4470: genSea.patch by Ticker
> > Berkin
> > 
> > Hi Gerd & Ticker,
> > 
> > Whilst I was testing Ticker's patch, Gerd's update was committed,
> > so
> > I am slightly behind the times. Both the patch and the committed
> > code
> > have solved almost all my coastline problems. Thanks for your
> > efforts. I now have no tiles that are being filled with sea, but I
> > do
> > have one tile that is now displaying the opposite problem, in that
> > it
> > now displays all land and no sea with both code versions, even
> > though
> > there is a significant amount of sea. Please see the attached
> > screenshot. I would have attached the tile, but it is rather large
> > for emailing.
> > 
> > Cheers,
> > Mike
> > 
> > -----Original Message-----
> > From: svn commit [mailto:svn at mkgmap.org.uk] 
> > Sent: 18 March 2020 18:23
> > To: mkgmap-svn at lists.mkgmap.org.uk; mkgmap-dev at lists.mkgmap.org.uk
> > Subject: [mkgmap-dev] Commit r4470: genSea.patch by Ticker Berkin
> > 
> > Version mkgmap-r4470 was committed by gerd on Wed, 18 Mar 2020
> > 
> > genSea.patch by Ticker Berkin
> > fixes flooded areas caused by changes in r4392 and other issues
> > with
> > unclosed coastlines 
> > 
> > http://www.mkgmap.org.uk/websvn/revision.php?repname=mkgmap&rev=447
> > 0
> > 
> > _______________________________________________
> > 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