<div dir="ltr"><div>Thanks for the votes and kind words :)<br></div><div><br></div><div>I'd applied Kurt's super-minor wording suggestions but haven't added any new sections (on fiona, extra examples, etc) yet. I'll try and incorporate those changes in the next couple days.</div><div><br></div><div></div><div>Cheers</div><div>Craig <br></div><div></div><div><br></div><div class="gmail_quote"><div dir="ltr">On Tue, 11 Dec 2018 at 11:07 Kurt Schwehr <<a href="mailto:schwehr@gmail.com" target="_blank">schwehr@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr">To follow on to Howard's comment about testing in python...<div><br></div><div>I'm looking forward to the day that GDAL can drop all python 2.x support and testing!</div><div><br></div><div>And anyone working on C++ testing is welcome to the code here.  I'll happily donate it with a license change to GDAL (it's apache 2.0 right now).  You are free refactor however.  Or just use any bits that you think are helpful if going to a different testing library.  I can also donate some of the 80+k files in my fuzzer corpus as test inputs that cover weird corners of the code.</div><div><br></div><div><a href="https://github.com/schwehr/gdal-autotest2/tree/master/cpp" target="_blank">https://github.com/schwehr/gdal-autotest2/tree/master/cpp</a><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 10, 2018 at 11:39 AM Howard Butler <<a href="mailto:howard@hobu.co" target="_blank">howard@hobu.co</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">+1<br>
<br>
I watched the ticket traffic and shuddered :) Thank you. GDAL's testing<br>
probably predates five or six Python testing regimes/eras.  I also have<br>
the concern about GDAL's testing going through Python, but this RFC will<br>
make it much easier for people to contribute and improve the story.<br>
<br>
<br>
Congratulations on a huge lift!<br>
<br>
<br>
Howard<br>
<br>
<br>
On 12/9/18 9:48 PM, Kurt Schwehr wrote:<br>
> I haven't had a chance to read the RFC yet, so I can't yet vote.  However,<br>
> a huge thank you to Craig and everyone else who put in effort to make this<br>
> happen!<br>
><br>
> On Sun, Dec 9, 2018 at 7:05 PM Craig de Stigter <<br>
> <a href="mailto:craig.destigter@koordinates.com" target="_blank">craig.destigter@koordinates.com</a>> wrote:<br>
><br>
>> Jonathan<br>
>><br>
>>> it's worth spending a little thought on coming up with a scheme for<br>
>> test-ids.<br>
>><br>
>> I've been through the list of parametrized tests and tweaked the `ids`<br>
>> kwargs to make them a little more helpful at first glance:<br>
>> <a href="https://github.com/OSGeo/gdal/pull/963/commits/8db599e7bc08b7dc73d81591898ed0f5f4243a58" rel="noreferrer" target="_blank">https://github.com/OSGeo/gdal/pull/963/commits/8db599e7bc08b7dc73d81591898ed0f5f4243a58</a><br>
>><br>
>> I didn't see any way to use `pytest_make_parametrize_id` really; IDs<br>
>> rightly vary enough between tests that I can't see that hook being very<br>
>> useful here.<br>
>><br>
>> Cheers<br>
>> Craig de Stigter<br>
>><br>
>> On Mon, 10 Dec 2018 at 09:59 jratike80 <<br>
>> <a href="mailto:jukka.rahkonen@maanmittauslaitos.fi" target="_blank">jukka.rahkonen@maanmittauslaitos.fi</a>> wrote:<br>
>><br>
>>> +0<br>
>>><br>
>>> -Jukka Rahkonen-<br>
>>><br>
>>><br>
>>> Even Rouault-2 wrote<br>
>>>> PSC members,<br>
>>>><br>
>>>> gentle reminder to cast your vote on this.<br>
>>>><br>
>>>> Thanks,<br>
>>>><br>
>>>> Even<br>
>>><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> Sent from: <a href="http://osgeo-org.1560.x6.nabble.com/GDAL-Dev-f3742093.html" rel="noreferrer" target="_blank">http://osgeo-org.1560.x6.nabble.com/GDAL-Dev-f3742093.html</a><br>
>>> _______________________________________________<br>
>>> gdal-dev mailing list<br>
>>> <a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
>>> <a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
>> _______________________________________________<br>
>> gdal-dev mailing list<br>
>> <a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
>> <a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> gdal-dev mailing list<br>
> <a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
> <a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
<br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/gdal-dev</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_976408271172400060m_5316469996167343448gmail_signature">--<div><a href="http://schwehr.org" target="_blank">http://schwehr.org</a></div></div>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/gdal-dev</a></blockquote></div></div>