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