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

Sat Mar 21 11:29:57 GMT 2020

Hi Mike, Gerd

I've also built 66973072.o5m from 
http://files.mkgmap.org.uk/download/463/66973072.zip

and don't get any of the messages about the coastline hitting tile edge
in the same direction.

With --generate-sea=multipolygon it look correct.

With --generate-sea=polygon it requires more in the style and .typ fileto get the land to show over the sea because this mode generates a polygon tagged natural=sea covering the full tile and then, for each but of land, a polygon tagged with --land-tag (default natural=land) 

Ticker

On Sat, 2020-03-21 at 08:44 +0000, Gerd Petermann wrote:
> Hi Mike,
> 
> I cannot reproduce the problem with r4473 and your file and option -
> -generate-sea=multipolygon.
> I can reproduce it with option --generate-sea=polygon, but that
> option is not recommended AFAIK.
> 
> Gerd
> 
> ________________________________________
> Von: mkgmap-dev <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> von Mike Baggaley <mike at tvage.co.uk>
> Gesendet: Freitag, 20. März 2020 23:18
> An: 'Development list for mkgmap'
> Betreff: Re: [mkgmap-dev] Commit r4470: genSea.patch by Ticker Berkin
> 
> Close inspection shows a very small coast outline of an outcrop at
> N50.45154 W3.55078 that may not be a valid polygon, another at
> N50.40129 W3.49404, and another at 50.39544 W3.51977, also, possible
> loops in the main coastline at N50.34285 W3.56060, N50.36633
> W3.57717, N50.36740 W 3.57671, N50.37806 W 3.58133, N50.34436
> W3.57435 and N50.33140 W3.56933. Also, what looks like a spur at
> N50.70210 W3.50376. The problem seems to depend on where the tile
> borders are - If I delete everything except the coastline data and
> process that through splitter and mkgmap, there are far fewer tiles
> and the result looks pretty well perfect as far as I can see.
> 
> Regards,
> Mike
> 
> 
> -----Original Message-----
> From: Mike Baggaley [mailto:mike at tvage.co.uk]
> Sent: 20 March 2020 21:35
> To: 'Development list for mkgmap' <mkgmap-dev at lists.mkgmap.org.uk>
> Subject: RE: [mkgmap-dev] Commit r4470: genSea.patch by Ticker Berkin
> 
> 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=4
> > > 47
> > > 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
> 
> 
> _______________________________________________
> 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