<div dir="ltr"><div>Yes it is. There was a commit on the test_gdal2tiles.py changing the checksum values for test 1 :)</div><div><br></div>that's odd, it's pyflakes that is throwing me those warnings... well flake8 actually that is pyflakes + pep8. I'll have a look anyway.<div>And yes, the style commits need to be separate, no worries.</div><div><br></div><div>I did launch the vagrant by commenting this missing script. Now what is the vagrant box supposed to do? my understanding is that is comes with gdal installed (from trunk?)</div><div>I think the gdal2tiles.py test (and maybe some other python ones) are not working in the vagrant box because those tests are looking for scripts to test (gdal2tiles.py for example) in relative from the test file test_gdal2tiles.py and because the Vagrantfile only maps the autotest folder from the host to the client VM, the source for those python scripts is not available in the VM.</div><div>Is it on purpose? would you see any issue mapping the source gdal folder inside the vagrant box next to the autotest folder?</div><div><br></div><div>Sorry for the questions, it's always hard to enter into a project.</div><div><br></div><div>Last one actually for this mail:</div><div>I have an issue on a particular image. To test a fix (and non-regression in the future) I can either try to do some mocking (but seeing how gdal2tiles.py is not very modular, that might prove a nightmare) or use some specific image that I know present the issue. If I want to test based on an image, is it an ok practice to put the image in the repo (I can see that there are several already but that doesn't seem like a great idea to me). Any other common storage place (AWS S3 or whatever) that you would have available or recommend otherwise?</div><div><br></div><div>Thanks</div></div><div class="gmail_extra"><br clear="all"><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>
<br><div class="gmail_quote">On Sun, Jan 22, 2017 at 11:09 AM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>
<div style="font-family:'monospace';font-size:9pt;font-weight:400;font-style:normal"><span class="">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">On dimanche 22 janvier 2017 09:00:59 CET Grégory Bataille wrote:</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> Hey all,</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> 2 questions:</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> seeing that it's complex to build trunk on mac (and that some recent</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> changes have an impact on gdal2tiles output),</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
</span><p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">is it ? You can have a look at the scripts in ci/travis/osx used for the OsX target on Travis-CI. They probably don't drag all dependencies though.</p><span class="">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> I wanted to setup the vagrant</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> box, but I get this error. Any clue?</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
</span><p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I've just committed the missing file. Was missing a svn add at some point during a branch merge.</p><span class="">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> Second question: is there a python style-guide?</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
</span><p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Not that much. <a href="https://trac.osgeo.org/gdal/wiki/rfc8_devguide" target="_blank">https://trac.osgeo.org/gdal/<wbr>wiki/rfc8_devguide</a> just mentions:</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">"""</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">* All Python code in autotest, swig/python/scripts and swig/python/samples should pass OK with the Pyflakes checker (version used currently: 0.8.1). This is asserted by Travis-CI jobs </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">* Python code should be written to be compatible with both Python 2 and Python 3. </p><span class="">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">"""</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> I can see some consistency</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> in the way the code is written but it's not following the PEP8 community</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> standard and my editor is growing red spots all over the place. Any concern</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">> if I try to correct that bit by bit to PEP8?</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
</span><p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I don't have a strong personal opinon about PEP8 vs non-PEP8, but it seems that many Python projects enforce PEP8 compliance, so why not. The only thing to be careful with is to separate pure style change commits from functional change commits. Otherwise it is a history mess.</p><span class="">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Even</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">-- </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Spatialys - Geospatial professional services</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><a href="http://www.spatialys.com" target="_blank">http://www.spatialys.com</a></p></span></div></blockquote></div><br></div>