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 <[email protected]> im Auftrag von Mike Baggaley <[email protected]> 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:[email protected]] Sent: 20 March 2020 21:35 To: 'Development list for mkgmap' <[email protected]> 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:[email protected]] Sent: 20 March 2020 09:45 To: Development list for mkgmap <[email protected]> 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:[email protected]] > > Sent: 19 March 2020 23:44 > > To: [email protected] > > 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:[email protected]] > > Sent: 18 March 2020 18:23 > > To: [email protected]; [email protected] > > 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 > > [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
