<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>probably no problem here.<br>My results:<br>PrecompiledSeaGenerator: no significant effect<br>BoundaryPreprocessor : very small effect. <br>I think in both cases the Element instances are not kept for long,<br>means, they are converted into something else <br>which is not based on Element. The number of Element instances<br>is not very high compared to Coord instances and / or the<br>doubles used in java areas.<br><br>Gerd<br><br><br><br><div>> Date: Wed, 21 Jan 2015 21:16:54 +0100<br>> From: wmgcnfg@web.de<br>> To: mkgmap-dev@lists.mkgmap.org.uk<br>> Subject: Re: [mkgmap-dev] echo improvements for elements with fake ids<br>> <br>> Hi Gerd,<br>> <br>> did you measure the impact on the BoundaryPreprocessor and the <br>> PrecompiledSeaGenerator?<br>> I guess they might load much more than 500k instances and I think it <br>> would be ok to disable the original id tracking when they are running.<br>> <br>> On the other hand can you roughly estimate the size of an additional <br>> tag? Is it the additional String instance?<br>> <br>> I am thinking how to fix the thing that the original id might be derived <br>> from a different element type. A tag would be easy to implement without <br>> big impacts.<br>> <br>> WanMil<br>> <br>> > Hi Mike,<br>> ><br>> > I've committed the patch as is. The memory impact is not that big as<br>> > we have typically less than 500.000 instances of Element in one tile,<br>> > so 8 more bytes * 500.000 ~ 4MB more heap.<br>> ><br>> > Gerd<br>> ><br>> > From: mike@tvage.co.uk<br>> > To: mkgmap-dev@lists.mkgmap.org.uk<br>> > Date: Mon, 19 Jan 2015 11:55:00 +0000<br>> > Subject: [mkgmap-dev] echo improvements for elements with fake ids<br>> ><br>> > The attached patch improves the echo and echotags messages by including the<br>> > id of the original element when a fake id is generated by mkgmap, thus<br>> > enabling you to find it in OSM.<br>> ><br>> > Please try, and commit if you are happy with the change.<br>> ><br>> > Regards,<br>> > Mike<br>> ><br>> ><br>> > _______________________________________________ mkgmap-dev mailing list<br>> > mkgmap-dev@lists.mkgmap.org.uk<br>> > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>> ><br>> ><br>> > _______________________________________________<br>> > mkgmap-dev mailing list<br>> > mkgmap-dev@lists.mkgmap.org.uk<br>> > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>> ><br>> <br>> _______________________________________________<br>> mkgmap-dev mailing list<br>> mkgmap-dev@lists.mkgmap.org.uk<br>> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br></div>                                            </div></body>
</html>