May 2018 - mkgmap-dev - The mkgmap lists

Commit r4192: correct check for "Error: Resolution value out of range 0-24: " message.
by svn commit 22 May '18

22 May '18

Version mkgmap-r4192 was committed by gerd on Tue, 22 May 2018 correct check for "Error: Resolution value out of range 0-24: " message. http://www.mkgmap.org.uk/websvn/revision.php?repname=mkgmap&rev=4192

1 0

Proposed change to splitter to make tile names unique
by Randolph J. Herber 20 May '18

20 May '18

Dear Sirs: I propose adding the following to uk.me.parabola.splitter.AreaList.java: *** AreaList-old.java 2019-05-20 11:28:45.739455200 -0500 --- AreaList.java 2018-05-11 20:01:47.861375200 -0500 *************** *** 280,286 **** cityFinder = new DefaultCityFinder(cities); } for (Area area : getAreas()) { ! area.setName(description); if (cityFinder == null) continue; --- 280,286 ---- cityFinder = new DefaultCityFinder(cities); } for (Area area : getAreas()) { ! area.setName(description + "-" + openLocationCode(area)); if (cityFinder == null) continue; *************** *** 350,354 **** --- 350,401 ---- } } + private static final String codes = "23456789CFGHJMPQRVWX"; + private static String openLocationCode(Area area) { + return openLocationCode( + Utils.toDegrees(area.getMinLat()), + Utils.toDegrees(area.getMinLong()), + Utils.toDegrees(area.getMaxLat()), + Utils.toDegrees(area.getMaxLong())); + } + + private static String openLocationCode( + double south,double west,double north,double east) { + double centerLatitude = 0.5 *(south + north)+90.; + double centerLongitude = 0.5 * (west + east)+180.; + double boxHeight = north - south; + double boxWidth = east - west; + double height = 400.; + double width = 400.; + String code = ""; + boolean done = false; + while(code.length() < 8 & !done) { + int p; + height /= 20.; + width /= 20.; + p = (int)Math.floor(centerLatitude/height); + code += codes.charAt(p); + centerLatitude -= p * height; + p = (int)Math.floor(centerLongitude/width); + code += codes.charAt(p); + centerLongitude -= p * width; + done = width <= boxWidth && height <= boxHeight; + } + if(!done) { + code += "+"; + while(!done) { + int la, lo; + height /= 5.; + width /= 4.; + la = (int)Math.floor(centerLatitude/height); + centerLatitude -= la * height; + lo = (int)Math.floor(centerLongitude/width); + centerLongitude -= lo * width; + code += codes.charAt(5*la+lo); + done = width <= boxWidth && height <= boxHeight; + } + } + return code; + } } The purpose is to give each tile an unique name by appending the Open Location Code of the largest region (shortest code) that fits within the tile. I have been using this code myself. https://en.wikipedia.org/wiki/Open_Location_Code Randolph J. Herber

1 0

Erroneous error message
by Randolph J. Herber 20 May '18

20 May '18

Dear Sirs: The code and the error message do not agree: if (value <= 0 || value > 24) throw new ExitException("Error: Resolution value out of range 0-24: " + s + " in levels specification " + levelSpec); Randolph J. Herber

1 0

Wrong address with addr:place
by Lorenzo Mastrogiacomi 20 May '18

20 May '18

I found this strange case of place mismatch on an object tagged with addr: place. The address of this point is: city: Recanati street/place: Frazione Montefiore housenumber: 23 https://www.openstreetmap.org/node/5469314272 This address gives no result but I found out that there are two results for this one: city: Recanati street/place: Contrada Mattonata housenumber: 23 the right one is here: https://www.openstreetmap.org/node/5468200919 but the other result points to the previous one. Here is a map in case anyone wants to try to understand what happened. https://drive.google.com/file/d/1mIdun-d7slpFz_RUwL-rhY-WALttkt4X/view? usp=drivesdk Lorenzo

1 0

Performance hints for Java JRE 9 + 10
by Gerd Petermann 16 May '18

16 May '18

Hi all, it seems that the new Garbage Collector (G1GC) that was introduced with Java JRE 9 causes longer run times for splitter and mkgmap. I've played with some options and found that the use of -XX:+AggressiveHeap seems to be a good idea when you use these newer JREs. Without this option the java programs tend to allocate more memory and thus the OS might start to swap. Gerd

4 3

Commit r4191: small DEM performance improvement: reduce number of double[][] allocations
by svn commit 16 May '18

16 May '18

Version mkgmap-r4191 was committed by gerd on Wed, 16 May 2018 small DEM performance improvement: reduce number of double[][] allocations Keep the 4x4 array that is filled for the bicubic interpolation to reduce GC strees http://www.mkgmap.org.uk/websvn/revision.php?repname=mkgmap&rev=4191

1 0

mkgmap 4189
by Steve Sgalowski 15 May '18

15 May '18

i just installed mkgmap 4189 , gave it a burl and this is the result java in use is 1.80.171 using mapuloader 4.894 Stephen

5 5

Commit r4190: Use special method to write 3 byte longitude values
by svn commit 15 May '18

15 May '18

Version mkgmap-r4190 was committed by gerd on Tue, 15 May 2018 Use special method to write 3 byte longitude values so that 8388608 (180) is written as -8388608 (-180). Patch by Ticker Berkin http://www.mkgmap.org.uk/websvn/revision.php?repname=mkgmap&rev=4190

1 0

Re: [mkgmap-dev] methods to write signed / unsigned integers
by Gerd Petermann 15 May '18

15 May '18

Hi all, I've now committed the patches for mkgmap and display tool. If you use display tool r531 or later please make sure to use the latest mkgmap version from trunk. Gerd ________________________________________ Von: mkgmap-dev <mkgmap-dev-bounces(a)lists.mkgmap.org.uk> im Auftrag von Steve Ratcliffe <steve(a)parabola.me.uk> Gesendet: Dienstag, 17. April 2018 23:31:56 An: mkgmap-dev(a)lists.mkgmap.org.uk Betreff: Re: [mkgmap-dev] methods to write signed / unsigned integers Hi Gerd, Ticker > sorry for that. I waited for a comment from Steve and forgot about it. > I'd still prefer that Steve gives an OK because he wrote most of the code that the patch changes. > I read through the patch and done some very simple tests and it looks OK to me. Hope someone has tested some really big maps. I think we should remove the commented out code and I noticed a few unneeded casts remaining (some maybe not actually as a result of the patch), and some javadoc errors. I've attached an incremental patch of the things I did, which you can take or leave as you wish. Lets get it committed. Its a big patch, thanks for going through all the cases and completing it. Steve

4 9

r4175 & canary-islands dont work
by Arndt Röhrig 14 May '18

14 May '18

3 7