<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>Hi WanMil,<br><br>with your patch I see more messages for my Niedersachsen data, <br>but not a single one starts with "Late invalid".<br><br>Maybe you are looking at an old result file?<br><br>Gerd<br><br><div>&gt; Date: Tue, 4 Feb 2014 21:21:31 +0100<br>&gt; From: wmgcnfg@web.de<br>&gt; To: mkgmap-dev@lists.mkgmap.org.uk<br>&gt; Subject: Re: [mkgmap-dev] Problem with turn restriction<br>&gt; <br>&gt; Hi Gerd,<br>&gt; <br>&gt; your fixes for 3) were already sorted out by my checks. So relations <br>&gt; without via coord in the bbox were not counted.<br>&gt; <br>&gt; I wonder why your fixes for 1) do not help. I have expected that it <br>&gt; removes some of the 851 problems...<br>&gt; <br>&gt; By the way:<br>&gt; 2) mmh, I don't mind adding that but I think it can be addressed with <br>&gt; very low priority. If there is a street where u-turns are not allowed <br>&gt; the street should be mapped with separate ways for each direction <br>&gt; (that's my opinion - don't know if that matches with the official <br>&gt; mapping guidelines).<br>&gt; <br>&gt; 4) Ok, they can be ignored. Would be great if we can detect them to <br>&gt; output different log messages for them.<br>&gt; <br>&gt; WanMil<br>&gt; <br>&gt; &gt; Hi WanMil,<br>&gt; &gt;<br>&gt; &gt; that's strange. With r3000 I saw many problems for Niedersachen, with<br>&gt; &gt; r3002 only 4, and those were the two examples.<br>&gt; &gt;<br>&gt; &gt; I am downloading latest Germany now.<br>&gt; &gt;<br>&gt; &gt; Gerd<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt; WanMil wrote<br>&gt; &gt;&gt; Hi Gerd,<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; I think your changes are good.<br>&gt; &gt;&gt; Anyhow for my special purpose I don't see any difference after your fixes.<br>&gt; &gt;&gt; So I will explain what and how I am checking the restrictions (see<br>&gt; &gt;&gt; attached patch with the check code).<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; All restrictions that are valid after loading but which cannot be<br>&gt; &gt;&gt; written to the map because there is a problem when the<br>&gt; &gt;&gt; RestrictionRelation.addRestriction(..) is called are logged with the<br>&gt; &gt;&gt; (not very useful...) text "Late invalid: "+URL of restriction.<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; See relation_problems.txt with the results. There are 851 problems in<br>&gt; &gt;&gt; Germany using the tiles created with attached areas.list.<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; I am not sure if really all logged restrictions are completely valid but<br>&gt; &gt;&gt; all I checked should make its way into the mkgmap compiled map.<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; I hope this helps you to find some other problematic places!<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; WanMil<br>&gt; &gt;&gt;<br>&gt; &gt;&gt;&gt; Hi WanMil,<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt; up to now I found these reasons for problems:<br>&gt; &gt;&gt;&gt; 1) one error in the branch: via coords were replaced without updating<br>&gt; &gt;&gt;&gt; the corrresponding restrictions and the hash map<br>&gt; &gt;&gt;&gt; 2) "no_u_turn" restrictions were not added<br>&gt; &gt;&gt;&gt; if from-way and to-way are equal. They are evaluated to be valid, but<br>&gt; &gt;&gt;&gt; I don't know if they really make sense?<br>&gt; &gt;&gt;&gt; 3) restrictions are added to the restrictions hash map even if the via<br>&gt; &gt;&gt;&gt; coord is<br>&gt; &gt;&gt;&gt; not contained in the bounding box.<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt; 4) restrictions that have a from-way or to-way which<br>&gt; &gt;&gt;&gt; is not added with a routable type or not at all,<br>&gt; &gt;&gt;&gt; e.g. way http://www.openstreetmap.org/way/225540783<br>&gt; &gt;&gt;&gt; has no tags but is part of three restriction relations.<br>&gt; &gt;&gt;&gt; Another example:<br>&gt; &gt;&gt;&gt; http://www.openstreetmap.org/relation/2880411<br>&gt; &gt;&gt;&gt; refers to ways that are tagged lane=tertiary<br>&gt; &gt;&gt;&gt; and the default style ignores them.<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt; In high-prec-coords branch r3003 I've fixed 1) to 3), please<br>&gt; &gt;&gt;&gt; check again.<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt; Gerd<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt;   &gt; Date: Mon, 3 Feb 2014 13:01:40 -0800<br>&gt; &gt;&gt;&gt;   &gt; From:<br>&gt; &gt;<br>&gt; &gt;&gt; gpetermann_muenchen@<br>&gt; &gt;<br>&gt; &gt;&gt;&gt;   &gt; To:<br>&gt; &gt;<br>&gt; &gt;&gt; mkgmap-dev@.org<br>&gt; &gt;<br>&gt; &gt;&gt;&gt;   &gt; Subject: Re: [mkgmap-dev] Problem with turn restriction<br>&gt; &gt;&gt;&gt;   &gt;<br>&gt; &gt;&gt;&gt;   &gt; WanMil wrote<br>&gt; &gt;&gt;&gt;   &gt; &gt;&gt; I want to make a small statistic why restriction relations become<br>&gt; &gt;&gt;&gt;   &gt; &gt;&gt; invalid. Maybe the problem is so seldom that it's not worthy...<br>&gt; &gt;&gt;&gt;   &gt; &gt;<br>&gt; &gt;&gt;&gt;   &gt; &gt; I have made a short stat with the high-prec branch:<br>&gt; &gt;&gt;&gt;   &gt; &gt; There are around 850 relations that are valid<br>&gt; &gt;&gt;&gt;   &gt; &gt; (RestrictionRelation.isValid() == true) after loading but that are<br>&gt; &gt;&gt;&gt; not<br>&gt; &gt;&gt;&gt;   &gt; &gt; valid when the StyledConverter calls<br>&gt; &gt;&gt;&gt;   &gt; &gt; RestrictionRelations.convertRelation(MapCollector ...).<br>&gt; &gt;&gt;&gt;   &gt; &gt; So it seems to me as if the problem is greater than expected.<br>&gt; &gt;&gt;&gt;   &gt;<br>&gt; &gt;&gt;&gt;   &gt; Yes, sounds too much. The only good case that I can think of<br>&gt; &gt;&gt;&gt;   &gt; is that the relation is saved by splitter because one of the<br>&gt; &gt;&gt;&gt;   &gt; related ways has at least one point within the boundary, but<br>&gt; &gt;&gt;&gt;   &gt; another part of the relation is outside of the boundary.<br>&gt; &gt;&gt;&gt;   &gt; If the via node is within the tile boundary we should be able<br>&gt; &gt;&gt;&gt;   &gt; to create the restriction.<br>&gt; &gt;&gt;&gt;   &gt;<br>&gt; &gt;&gt;&gt;   &gt; Gerd<br>&gt; &gt;&gt;&gt;   &gt;<br>&gt; &gt;&gt;&gt;   &gt;<br>&gt; &gt;&gt;&gt;   &gt;<br>&gt; &gt;&gt;&gt;   &gt; --<br>&gt; &gt;&gt;&gt;   &gt; View this message in context:<br>&gt; &gt;&gt;&gt; http://gis.19327.n5.nabble.com/Problem-with-turn-restriction-tp5795049p5795167.html<br>&gt; &gt;&gt;&gt;   &gt; Sent from the Mkgmap Development mailing list archive at Nabble.com.<br>&gt; &gt;&gt;&gt;   &gt; _______________________________________________<br>&gt; &gt;&gt;&gt;   &gt; mkgmap-dev mailing list<br>&gt; &gt;&gt;&gt;   &gt;<br>&gt; &gt;<br>&gt; &gt;&gt; mkgmap-dev@.org<br>&gt; &gt;<br>&gt; &gt;&gt;&gt;   &gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;&gt; _______________________________________________<br>&gt; &gt;&gt;&gt; mkgmap-dev mailing list<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;<br>&gt; &gt;&gt; mkgmap-dev@.org<br>&gt; &gt;<br>&gt; &gt;&gt;&gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>&gt; &gt;&gt;&gt;<br>&gt; &gt;&gt;<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; _______________________________________________<br>&gt; &gt;&gt; mkgmap-dev mailing list<br>&gt; &gt;<br>&gt; &gt;&gt; mkgmap-dev@.org<br>&gt; &gt;<br>&gt; &gt;&gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; relation_check.patch (1K)<br>&gt; &gt;&gt; &amp;lt;http://gis.19327.n5.nabble.com/attachment/5795291/0/relation_check.patch&amp;gt;<br>&gt; &gt;&gt; areas.list (14K)<br>&gt; &gt;&gt; &amp;lt;http://gis.19327.n5.nabble.com/attachment/5795291/1/areas.list&amp;gt;<br>&gt; &gt;&gt; relation_problems.txt (71K)<br>&gt; &gt;&gt; &amp;lt;http://gis.19327.n5.nabble.com/attachment/5795291/2/relation_problems.txt&amp;gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt; --<br>&gt; &gt; View this message in context: http://gis.19327.n5.nabble.com/Problem-with-turn-restriction-tp5795049p5795294.html<br>&gt; &gt; Sent from the Mkgmap Development mailing list archive at Nabble.com.<br>&gt; &gt; _______________________________________________<br>&gt; &gt; mkgmap-dev mailing list<br>&gt; &gt; mkgmap-dev@lists.mkgmap.org.uk<br>&gt; &gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>&gt; &gt;<br>&gt; <br>&gt; _______________________________________________<br>&gt; mkgmap-dev mailing list<br>&gt; mkgmap-dev@lists.mkgmap.org.uk<br>&gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br></div>                                               </div></body>
</html>