<div dir="ltr"><div dir="ltr">On Thu, Oct 29, 2020 at 3:15 PM Even Rouault <<a href="mailto:even.rouault@spatialys.com">even.rouault@spatialys.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Sean,<br>
<br>
> In rasterio's tests I have noticed a change in the order of coordinates in<br>
> the interior rings of polygons extracted by GDAL's polygonizer and a change<br>
> in the number of polygons extracted, but no other issues. I don't think<br>
> these need to hold up the release.<br>
<br>
Hum, changes in the order of coordinates are expected due to a change in the <br>
algorithm. GDAL autotest had to be adapted for that<br>
<a href="https://github.com/OSGeo/gdal/commits/c3a6bf99e3fd7fec4e11f9d1910e200b82742a5d" rel="noreferrer" target="_blank">https://github.com/OSGeo/gdal/commits/c3a6bf99e3fd7fec4e11f9d1910e200b82742a5d</a> <br>
. (testing with GEOSEquals_r() would be even more robust)<br>
But I wouldn't have necessarily expected a change in the number of polygons. <br>
Does the new result look correct ?<br>
<br>
Even<br></blockquote><div><br></div><div>I misread my pytest output. It was only the JSON formatting of the polygon that changed, the number of polygons is unchanged. Sorry for the false alarm.</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Sean Gillies</div></div></div>