<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 Anor,<br><br>I've tested this relation. It seems to work fine with the v2 of the patch.<br>You should see a message like this:<br>uk.me.parabola.mkgmap.reader.osm.POIGeneratorHook&nbsp; e:\testdata\rel4681234.osm: using label node as location for POI for rel 4681234 Progresso at -26.972253/-49.075171<br><br>Gerd<br><br><div><hr id="stopSpelling">From: anorcarlos@hotmail.com<br>To: mkgmap-dev@lists.mkgmap.org.uk<br>Date: Thu, 21 May 2015 09:35:14 -0300<br>Subject: Re: [mkgmap-dev] [Patch v2] use role=label member to place POI<br><br>

<style><!--
.ExternalClass .ecxhmmessage P {
padding:0px;
}

.ExternalClass body.ecxhmmessage {
font-size:12pt;
font-family:Calibri;
}

--></style>
<div dir="ltr"><br id="ecxFontBreak"><span id="ecxresult_box" lang="en"><span class="ecxhps">Hi</span> <span class="ecxhps">Gerd</span><br><br> <span class="ecxhps">The error</span> <span class="ecxhps">did not yield</span> <span class="ecxhps">but following the</span> </span><span id="ecxresult_box" lang="en"><span class="ecxhps">example</span></span><span id="ecxresult_box" lang="en"><span class="ecxhps"></span> <span class="ecxhps">this relationship&nbsp; http://www.openstreetmap.org/relation/4681234#map=15/-26.9736/-49.0721 <br><br></span> <span class="ecxhps"></span><span class="ecxhps">doubles</span> <span class="ecxhps">this point.<br><br></span></span><br>&nbsp;&nbsp;&nbsp; []<br>Anor &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; <br><font style="" color="#ff0000"></font>&nbsp;<br><font style="" color="#ff0000"></font>&nbsp;<br><font style="" color="#ff0000"></font>&nbsp;<br><font style="" color="#ff0000"></font>&nbsp;<br><font style="" color="#ff0000"></font>&nbsp;<br><font style="" color="#ff0000"></font>&nbsp;<br><br><font style="" color="#ff0000"><font style="background-color:#92d050;"><u>_______________________________________________________________________________________________________</u></font></font><div><font color="#ff0000"><font style="background-color:#92d050;"><u><br></u></font></font>
<p align="left"><font face="Arial"><font size="5"><font style="background-color:#ffffff;" color="#205867"><strong>Anor C. A. de Souza &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;Co</strong></font><strong><font style="background-color:#ffffff;" color="#ff0000"><font color="#205867">ncórdia&nbsp;SC&nbsp;</font></font></strong></font></font></p><p align="left"><font face="Arial"><font size="5"><strong><font style="background-color:#ffffff;" color="#ff0000"><font color="#205867">&nbsp;&nbsp;</font></font></strong></font></font></p><br><p align="left">&nbsp;</p>
<p align="center"><strong><font color="#ff0000"></font></strong>&nbsp;</p>&nbsp;<br>&nbsp;<br>&nbsp;<br>&nbsp;<br>&nbsp;<br>&nbsp;<br>&nbsp;<br>&nbsp;<br></div><br><br><div><hr id="ecxstopSpelling">From: gpetermann_muenchen@hotmail.com<br>To: mkgmap-dev@lists.mkgmap.org.uk<br>Date: Thu, 21 May 2015 09:30:47 +0200<br>Subject: [mkgmap-dev] [Patch v2] use role=label member to place POI<br><br>

<style><!--
.ExternalClass .ecxhmmessage P {
padding:0px;
}

.ExternalClass body.ecxhmmessage {
font-size:12pt;
font-family:Calibri;
}


--></style>
<div dir="ltr">Hi all,<br><br>sorry, v1 was missing a check for null , attached is v2, the binary is here:<br><code><a href="http://files.mkgmap.org.uk/download/269/mkgmap.jar" target="_blank">http://files.mkgmap.org.uk/download/269/mkgmap.jar</a><br><br></code>To ease testing I've added a log message with error level ,e.g. <br><br><font style="font-size:12pt;" size="3">...&nbsp; <b>using label node as location</b> for POI for rel 334443 Goiás at -15.933471/-50.140561</font><font style="font-size:12pt;" size="3"><br></font><br>it will also complain loudly when the node is not used because the names don't match:<br>... rel http://www.openstreetmap.org/relation/331515 ,<b>node with role label is ignored because it has a different name<br><br></b>Please suggest better text for the messages.<br>These messages show were the patch changes something, it will be <br>changed to debug/warn level before committing the patch.<br><br>If I got that right, Marcio and Anor propose to always use the label node<br>to place the generated POI for admin_level=10 boundaries,&nbsp; <br>but I see one potential problem here:<br>The node might also be used by different relations, and in that case<br>we will produce different POI at the same location.<br>I think this is something we want to avoid.<br><br>One more point I am not sure about:<br>The node in the mp-relation is only used when both the relation and the node have a name tag<br>(or whatever the name-tag-list tells mkgmap to use as name), <br>and when these are equal.<br>Do you see a case where we should use the node even if one of the name tags is missing?<br><br>Gerd<br>                                               </div>
<br>_______________________________________________
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev</div>                                               </div>
<br>_______________________________________________
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev</div>                                               </div></body>
</html>