<div dir="ltr"><div>Hi Sebastiaan,</div><div><br></div><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 6 avr. 2020 à 13:39, Sebastiaan Couwenberg <<a href="mailto:sebastic@xs4all.nl">sebastic@xs4all.nl</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Thanks for sharing your software.<br>
<br>
Looking through the source I noticed some code smells like creating XML<br>
as strings instead of using lxml and not quoting variables used in SQL.<br>
flake8 also reports several issues with convert2osm.py.<br></blockquote><div> </div><div>Thank you for your time and these issues.</div><div>We'll certainly fix this in a next relase, at least for SQL.</div><div>Regarding Python, ogr2osm may replace it as discussed below.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
One also wonders how this compares to ogr2osm which can be used for the<br>
same purpose. Have you tried using that before deciding to write this?<br></blockquote><div><br></div><div>At the time we have started to set up our implementation, ogr2osm didn't allow to manage how connections are done.</div><div>We had need to prevent some geometries to connect and that's why additional tags like pgsql2osm:layer exists.</div><div>Because this kind of information isn't contained in geometries actually.<br></div><div><br></div><div>As I understand how ogr2osm works (maybe mistaken), it splits geometries in small segments and then connect them depending on overlapping nodes.</div><div>We also had a need to preserve geometries shapes.</div><div><br></div><div>However, that would worth a try to use ogr2osm instead of our python script to convert data once topology produced according to 2 needs upside.</div><div><br></div><div>All the best</div><div><br></div><div>François<br></div><div><br></div><div><br></div><div><br></div></div></div></div>