<div dir="ltr">Hi,<div><br></div><div>Thoughts that I have:</div><div><br></div><div> - this is a cool thing that I've needed multiple times. I worked around its absence by fiddling with ST_MakeLine(ST_Collect(line, ST_OffsetCurve(line, 1))) and then trying to make result valid, which sometimes worked;</div><div> - how do I remember which way is it being offset? I expect ST_Buffer(ST_Boundary(ST_ForceRHR(polygon)), 10, 'single_side') to be buffered "outside". Does it hold true?<br></div><div> - if there are no better ideas on ST_Buffer signature I'd say your option is good enough for me;</div><div> - there's no error reporting mentioning `single_side` as valid option;<br></div><div> - can `single_side=1` also work for `single_sided=1` and just `single_side`/`single_sided`?</div><div> - what does it do for line and for point?</div><div> - does it work on EMPTY, LINESTRING(0 0, 0 0), MULTILINESTRING, and self-intersecting one?</div><div> - how do I reverse the direction of buffering? I understand there's negative buffer parameter, does it work? I see you parse parameter to be single_side=1/0, maybe -1 will also make sense? </div><div> - out of curiosity, will ST_Buffer(geom,0,'single_side') fix invalid geometries? :)<br><br>As someone responsible for github in PostGIS, I encourage you to put it to GitHub as a PR: that'll get you test coverage report, travis CI greenlight, a way to comment on your patch inline, and usual PR merge time on it recently improved. :)</div><br><div class="gmail_quote"><div dir="ltr">чт, 18 янв. 2018 г. в 12:56, Stephen Knox <<a href="mailto:stephenknox73@gmail.com" target="_blank">stephenknox73@gmail.com</a>>:<br></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi List,<div><br></div><div>I noticed that PostGIS didn't implement the single-sided buffer functionality of Geos (NB. not the Offset Curve functionality, but a polygon), so have submitted a patch to implement that: <a href="https://trac.osgeo.org/postgis/ticket/3989" target="_blank">https://trac.osgeo.org/postgis/ticket/3989</a> </div><div><br></div><div>I have refrained from adding tests and documentation until I get some agreement that the function signature is acceptable, as I am a new contributor, and I think the ST_Buffer signature is already quit "unusual" in that it implements text parameters.</div><div><br></div><div>I was hoping that post the recent patch releases, a more experienced contributor might be able to look at my (small) patch and give some feedback.</div><div><br></div><div>I can put it up on Github as well if that would help.</div><div><br></div><div>Many thanks</div><div><br>Stephen</div></div>
_______________________________________________<br>
postgis-devel mailing list<br>
<a href="mailto:postgis-devel@lists.osgeo.org" target="_blank">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/mailman/listinfo/postgis-devel</a></blockquote></div></div>