Re: [mkgmap-dev] HGT - getElevation()

2018-01-18 Thread Gerd Petermann
Hi Andrzej, I'd prefer to use unmodified dem-dist values for now, besides that the unit tests don't compile with it. After adding 0.01 as new parm some tests fail. Probably not so important, as the unit test HGTConverterTest is bad anyway, it tests non-public methods and your patch shows that

Re: [mkgmap-dev] HGT - getElevation()

2018-01-18 Thread Andrzej Popowski
Hi Gerd, I have prepared patch according to these guessed rules. It rounds dem-dist and shift coordinates of top-left corner of DEM areas. Since dem-dist doesn't correspond exactly to HGT, interpolation is always active. I think there could be 2 ways to improve DEM quality. Interpolation

Re: [mkgmap-dev] HGT - getElevation()

2018-01-18 Thread Andrzej Popowski
Hi Gerd, I have executed your script and looked at maps. I have tested them with artificial HGT too. Only the original-unaligned map has DEM at wrong position. It is moved by about 0.00033 degree right and down. I looked at offset calculated as (DEM coordinate)%(dem-dist) and only for this

Re: [mkgmap-dev] splitter r585 implements new option align-for-dem

2018-01-18 Thread Gerd Petermann
Hi Henning, please try r586, see http://www.mkgmap.org.uk/websvn/revision.php?repname=splitter=586 Gerd Von: mkgmap-dev im Auftrag von Henning Scholland Gesendet: Donnerstag, 18. Januar 2018

Re: [mkgmap-dev] splitter r585 implements new option align-for-dem

2018-01-18 Thread Henning Scholland
Hi Gerd I imported both kml-files to josm and added some taggs to make it more clear. landuse=grass -> split result based on r584 and also roughly same as polygon used by splitter landuse=industrial -> result with r585 and alignment to DEM. These tiles should be in templates args landuse

Re: [mkgmap-dev] splitter r585 implements new option align-for-dem

2018-01-18 Thread Gerd Petermann
Hi Henning, yes, I also see such differences. Looking at it now... Gerd Von: mkgmap-dev im Auftrag von Henning Scholland Gesendet: Donnerstag, 18. Januar 2018 16:28:16 An:

Re: [mkgmap-dev] splitter r585 implements new option align-for-dem

2018-01-18 Thread Henning Scholland
Hi Gerd I imported both kml-files to josm and added some taggs to make it more clear. landuse=grass -> split result based on r584 and also roughly same as polygon used by splitter landuse=industrial -> result with r585 and alignment to DEM. These tiles should be in templates args landuse

Re: [mkgmap-dev] splitter r585 implements new option align-for-dem

2018-01-18 Thread Henning Scholland
Sorry, I forgotten to add this: It seems the reason is that --polygon-desc-file is not working well any more. So main reason is, that the map is getting bigger. You can see it in last mails attached kml-files. Henning On 18.01.2018 22:33, Henning Scholland wrote: > Hi Gerd, > I have quite bad

Re: [mkgmap-dev] splitter r585 implements new option align-for-dem

2018-01-18 Thread Gerd Petermann
Hi Henning, this is not surprising for me. Tiles should be a bit different, if you look at them in JOSM you have to zoom to maybe 30 m. Why does the size grow? The effect of the alignment is higher precision, means, less interpolation. With interpolation, the values are averaged, so deltas are

Re: [mkgmap-dev] r4070 in dem-tdb branch

2018-01-18 Thread Henning Scholland
Hi Gerd, thanks for merging it. Hi Steve, I tried it on maps, where I previously had problems with block size. Now these maps compile well without increasing block size manually. Well done! Henning On 18.01.2018 19:14, Gerd Petermann wrote: > Hi all, > > I've merged branch auto-block into r4070

[mkgmap-dev] r4070 in dem-tdb branch

2018-01-18 Thread Gerd Petermann
Hi all, I've merged branch auto-block into r4070 in the dem-tdb branch. I've first reverted the block-size.patch and fixed two conflicts manually. Gerd ___ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk

Re: [mkgmap-dev] Automatic block-size setting

2018-01-18 Thread Gerd Petermann
Hi Steve, I think best would be to ignore the --block-size option, at least for a for a while like we do with --remove-short-arcs. This makes it easier to compare results of two different binaries. Gerd Von: mkgmap-dev

Re: [mkgmap-dev] splitter r585 ; dem-dist 3342 or 3314 ?

2018-01-18 Thread Gerd Petermann
Hi Thomas, oops, sorry, 3342 was a typo, should be 3314 ~ 2^32 / (360 * 3600) and 9942 ~ 2^32 / (360*1200) Gerd Von: mkgmap-dev im Auftrag von Thomas Morgenstern Gesendet: Donnerstag, 18.

Re: [mkgmap-dev] splitter r585 ; dem-dist 3342 or 3314 ?

2018-01-18 Thread Thomas Morgenstern
Hi Gerd, in some posts are written different -dem-dist . Which is the right, 3314 or 3342 ? For using with --align-for-dem 1" hgt ? thomas -- Von: "Gerd Petermann" Datum: Mittwoch, 17. Januar 2018 17:42 An: