<div dir="ltr"><div dir="ltr">Hi,<div><br></div><div>I implemented support for 3D solid POLYHEDRALSURFACE and 3D solid TIN on lwgeom side.</div><div>Implementation of three dimensional ST_3DIntersection / ST_3DDistance I believe is now feature-match with SFCGAL, so I removed SFCGAL switch without any renamings. </div><div><br></div><div>This way I removed most of sfcgal tests, which are in turn are just core tests with backend switch on top.</div><div><br></div><div>There are things in postgis_sfcgal that really all not about SFCGAL, like ST_MakeSolid or ST_IsSolid - they can be moved out of it as they only check flags.</div><div><br></div><div>Anybody willing to split up the .so files now that the math is fixed?</div><div><a href="https://trac.osgeo.org/postgis/ticket/4258">https://trac.osgeo.org/postgis/ticket/4258</a> <br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Dec 1, 2018 at 1:59 PM Darafei "Komяpa" Praliaskouski <<a href="mailto:me@komzpa.net">me@komzpa.net</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"><div dir="ltr">I've started removing SFCGAL function clones.<div><br></div><div>ST_Area is committed to trunk, ST_Distance is similarly going to be <a href="https://github.com/postgis/postgis/pull/347/files" target="_blank">https://github.com/postgis/postgis/pull/347/files</a> <br><br>Overlay functions to follow. If you have any precious query example that does not work without SFCGAL, it's really time to get it into test suite (or at least this thread), before those get removed and "we never promised *that* will work". I've checked 2D TINs on ST_Area manually and it worked even on my 2.5. On the other hand, SFCGAL just barfs on CIRCULARSTRING, so removing SFCGAL there seems to *add* features.</div></div><br><div class="gmail_quote"><div dir="ltr">ср, 28 нояб. 2018 г. в 18:28, Regina Obe <<a href="mailto:lr@pcorp.us" target="_blank">lr@pcorp.us</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> It looks fine to me as well.<br>
> <br>
> I also agree we better find a semantic difference between function names,<br>
> rather than just adding a "SFCGAL" prefix or suffix.<br>
> <br>
> The main difference is that SFCGAL handles TIN/PolyhedralSurfaces and is<br>
> able to work with them either as meshes or as volumes (if the mesh is closed).<br>
> <br>
> So for functions that overlap and are handled by the backend switch that we<br>
> would like to avoid, what about something like ST_3DIntersection() that<br>
> becomes ST_3DMeshIntersection() + ST_3DVolumeIntersection() in<br>
> postgis_sfcgal.so ? With similar prefix "mesh" and "volume" for union,<br>
> difference, intersects.<br>
> <br>
[Regina Obe] <br>
I like that idea.  ST_3DIntersection I wasn't so concerned with because PostGIS core doesn't have that at all,<br>
Though I like the semantic difference between the two.<br>
<br>
Maybe we can mark ST_3DIntersection as deprecated<br>
<br>
> About 2D intersection handled by SFCGAL, the only added value is the<br>
> handling of TIN/PS. But do we really need such things in 2D only ?<br>
> <br>
[Regina Obe] <br>
That's what I thought the difference was too and I would say probably not.<br>
I can't think of a reason why people want to manage 2D TIN / 2D polyhedralsurfaces just cause they exist.<br>
<br>
> There is also a SFCGAL ST_Area(), but I can't remember what it adds ...<br>
[Regina Obe] <br>
I haven't tried, maybe it works for TINs where as the PostGIS one definitely doesn't.<br>
<br>
_______________________________________________<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>-- <br><div dir="ltr" class="gmail-m_-6409786561279675254gmail_signature"><div dir="ltr">Darafei Praliaskouski<br>Support me: <a href="http://patreon.com/komzpa" target="_blank">http://patreon.com/komzpa</a></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div>Darafei Praliaskouski</div><div>Support me: <a href="http://patreon.com/komzpa" target="_blank">http://patreon.com/komzpa</a></div></div></div></div>