[mkgmap-dev] Routing problem on Garmin Oregon/edge with default style (and other styles)

Tue Jul 7 09:38:56 BST 2020

Hi all,

I am back home and started to look at the data. I assume the special case with way https://www.openstreetmap.org/way/27866666/history#map=16/40.4820/-3.9354 is that it is not connected to other roads in both end nodes AND that is is connected to multiple other roads between the ends. I guess I didn't think of this case before.

I can reproduce the crash with NodCheck when I create a map for 64130028.osm.pbf using only the option --route, but up to now I was not able to reproduce it with a smaller input file just containing the area around the special way, so there might be a 2nd problem...

Gerd

________________________________________
Von: mkgmap-dev <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag von Gerd Petermann <gpetermann_muenchen at hotmail.com>
Gesendet: Mittwoch, 1. Juli 2020 11:14
An: Development list for mkgmap
Betreff: Re: [mkgmap-dev] Routing problem on Garmin Oregon/edge with default style (and other styles)

Hi all, the patch reverts one of the major size improvements of the branch. It adds again routing nodes at the end of all roads. Seems there are special cases. Will look at them again when I am back.
Gerd

---- Steve Ratcliffe schrieb ----

Hi

 > Update, previous version had some errors, conclusion is the same

OK, Thanks that is great.

So I don't think that there wasn't really intended to be a change to
routing in that branch, but there was an experimental option that was
added, and then removed, with a couple of lines of code omitted,
probably unintentionally.  This is what causes the failure in NodCheck
and maybe causes the failure on the device too.

The attached patch restores the functionality and NodCheck does not
crash on the result

There is a pre-compiled jar there:
http://files.mkgmap.org.uk/download/484/mkgmap.jar

It may interact or interfere with the routing island removal code that
was also made in that branch.  I've no idea if it does or not, it is
just something to check.

Steve