On Sun, Aug 11, 2013 at 3:35 PM, Jason Remillard <span dir="ltr"><<a href="mailto:remillard.jason@gmail.com" target="_blank">remillard.jason@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


The software is new, and I am hoping to focus the review on it, rather<br>
than a tagging discussion.</blockquote><div><br></div><div>The script looked reasonable, and was easy to read.</div><div><br></div><div>Note that you can (now) supply changeset tags to JOSM in the XML, removing one more source of human error during script execution.</div>

<div><br></div><div>I'm glad you propose to keep the <span style="background-color:rgb(255,255,255);font-family:sans-serif;font-size:13px;line-height:19.1875px">massgis::PALID_ID</span></div><div><br></div><div><span style="background-color:rgb(255,255,255);font-family:sans-serif;font-size:13px;line-height:19.1875px">I'd suggest counting any nodes with building=yes area=yes that are NOT from the "</span><font face="sans-serif"><span style="line-height:19.1875px">MassGIS Buildings" import, and seeing if those are practical to manually fix.</span></font></div>

</div>