<div dir="ltr">Hello,<div><br></div><div>If I want to really spend some time for this utility, I'd like to do some clean engineering around it.</div><div>What I'd like to do is:</div><div><br></div><div>- split the script into several files (smaller, single purpose, ...)</div><div>--> can you confirm that on all target platform, all .py files are deployed and the /usr/local/bin/gdal2tiles.py is just an hyperlink on the source file installed somewhere else?</div><div>--> do you see any issue with that?</div><div><br></div><div>- Introduce a standard unittesting platform - pytest + tox I guess - (on top of the autotest thing, I would not attempt to redo them all) that would be incorporated in the CI</div><div><br></div><div>- Clean up the code: follow PEP8 and fix pylint issues. Put PEP8 and pylint in the CI. Rename the hell out of the variables because I'm banging my head with rx, ry, lrux and the like :)</div><div><br></div><div>Any comments, warnings, problems you can think of?</div><div><br></div><div>Cheers</div><div><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><font color="#999999"><br></font></div><div><font color="#999999">---</font></div><div><font color="#999999">Gregory Bataille</font><br></div></div></div></div>
</div></div>