<div dir="ltr"><div>It's not intentional. I will fix it in GEOS and add a test there. You might consider removing the PostGIS test because it's only testing GEOS behavior.</div><div><br></div><div>Dan<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 21, 2022 at 10:35 AM Regina Obe <<a href="mailto:lr@pcorp.us">lr@pcorp.us</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">Hi all,<br>
<br>
PostGIS geos310 regression test, detailed here:<br>
<br>
<a href="https://trac.osgeo.org/postgis/ticket/5260" rel="noreferrer" target="_blank">https://trac.osgeo.org/postgis/ticket/5260</a><br>
<br>
Started failing around October 18th.<br>
<br>
The test is this:<br>
SELECT '#168', ST_NPoints(g), ST_AsText(g), ST_isValidReason(g)<br>
FROM ( VALUES<br>
('01060000C00100000001030000C00100000003000000E3D9107E234F5041A3DB66BC97A30F<br>
4122ACEF440DAF9440FFFFFFFFFFFFEFFFE3D9107E234F5041A3DB66BC97A30F4122ACEF440D<br>
AF9440FFFFFFFFFFFFEFFFE3D9107E234F5041A3DB66BC97A30F4122ACEF440DAF9440FFFFFF<br>
FFFFFFEFFF'::geometry)<br>
) AS v(g);<br>
<br>
<br>
I'm guessing might be caused by this commit -<br>
<a href="https://github.com/libgeos/geos/commit/32348a68c5212c89cfefab46891d2b3aada4ab40" rel="noreferrer" target="_blank">https://github.com/libgeos/geos/commit/32348a68c5212c89cfefab46891d2b3aada4a<br>
b40</a> <br>
which happened around the same time tests started failing.<br>
<br>
The difference from prior versions is that the 3.12 (main branch) now emits<br>
for isValidReason one less coordinate<br>
<br>
Too few points in geometry component[4275341.96977851 259186.966993061]<br>
<br>
Instead of:<br>
Too few points in geometry component[4275341.96977851 259186.966993061<br>
1323.76295828331]<br>
<br>
<br>
Which to me makes sense if we are only considering 2 dimensions for<br>
validity.<br>
<br>
Before I change the test, I want to confirm that this is an intentional<br>
change.<br>
<br>
Thanks,<br>
Regina<br>
<br>
<br>
<br>
_______________________________________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/geos-devel</a><br>
</blockquote></div>