Attachments (1)
Change History (11)
comment:1 by , 12 years ago
Milestone: | PostGIS 2.1.0 → PostGIS Future |
---|
by , 12 years ago
Attachment: | triboundary.patch added |
---|
comment:2 by , 12 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:4 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
committed at r11358 . If you can get in a patch for TIN before end next week, I might be able to commit (and call this fully supporting tins in docs). Right now (I'd have to put in just a note that it supports triangle, which ain't bad since I think ST_Dump supports and they can do their own boundary by combining the 2).
comment:5 by , 12 years ago
Type: | enhancement → patch |
---|
comment:7 by , 12 years ago
Keywords: | history added |
---|
comment:8 by , 12 years ago
I'm seeing too many changes in lwgeom-geos connectors these days. May be worth some talk on the mailing list before it messes up too much ? Do I understand correctly that TRIANGLES are being threated like MULTILINESTRINGS when converted to GEOS ? Would it make more sense to convert to polygon ? Should we expose a Triangle geometry in GEOS C-API ?
comment:9 by , 12 years ago
I don't think that triangles are converted to geos at all. A look through the LWGEOM2GEOS() function doesn't seem to have any entry for a TRIANGLETYPE in the switch, so I think you'd get a call to lwerror() if you somehow managed to pass in a triangle. Curved geometries will fail also. Perhaps something else is converting them before a call to LWGEOM2GEOS().
I don't think GEOS has a triangle type, so you'd have to convert to a polygon if you were to do this.
The attached patch will implement this. A polygon would be pretty simple to do without going to GEOS also, though not trivial like the triangle.