<div dir="ltr">OK, so some alternatives I see are:<div><br></div><div>1) rename the functions to more clearly indicate that they only consider polygons (ST_ForcePolygonCW, ST_IsPolygonCW ? )<br><div class="gmail_extra">2) modify new functions and ST_ForceRHR to also consider orientation of closed LineStrings </div><div class="gmail_extra">3) modify new functions to also consider closed LineStrings, without modifying ST_ForceRHR</div><div class="gmail_extra"><br></div><div class="gmail_extra">I don't like the idea of modifying ST_ForceRHR, since it has existed forever, and modifying this would also suggest changes to ST_ForceLHR, whose implementation is contained within SFCGAL.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I'm inclined towards #1.  Do people have any thoughts on these alternatives, or maybe a different solution?</div><div class="gmail_extra"><br></div><div class="gmail_extra">Dan</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 8, 2017 at 6:26 AM, Sandro Santilli <span dir="ltr"><<a href="mailto:strk@kbt.io" target="_blank">strk@kbt.io</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Feb 07, 2017 at 07:55:27PM -0500, Daniel Baston wrote:<br>
<br>
> Non-polygonal geometries, including closed LineStrings, are<br>
> simply passed through.<br>
<br>
</span>This is the part I don't like. A closed LineString is really the<br>
only geometry that can be strictly clockwise or clounterclockwise<br>
(while polygons want shells and hole in opposite directions) so<br>
I find it counter-intuitive for ST_IsCW and ST_IsCCW to both return<br>
true for them, and for ST_ForceCW and ST_ForceCCW to return them<br>
untouched.<br>
<br>
--strk;<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
postgis-devel mailing list<br>
<a href="mailto:postgis-devel@lists.osgeo.org">postgis-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/postgis-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/postgis-devel</a></div></div></blockquote></div><br></div></div></div>