Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#570 closed defect (fixed)

Add Z support in delaunay triangulation

Reported by: strk Owned by: geos-devel@…
Priority: major Milestone: 3.4.0
Component: Default Version: master
Severity: Unassigned Keywords: history
Cc: blc56

Description

Following up #487 (comment:14) this ticket is to add Z support to Delaunay triangulation

Attachments (2)

delaunay_2_5d.patch (1008 bytes) - added by blc56 7 years ago.
delaunay_2_5d.2.patch (3.8 KB) - added by blc56 7 years ago.

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by blc56

Attachment: delaunay_2_5d.patch added

comment:1 Changed 7 years ago by blc56

Patch attached.

comment:2 Changed 7 years ago by blc56

Also, I can confirm that JTS does preserve the Z value.

comment:3 Changed 7 years ago by strk

I didn't test the patch but I've the impression it would always add a Z ? I think the output should only have a Z when the input does. Also, could you add tests for this ? Thanks !

comment:4 Changed 7 years ago by blc56

Yup. I agree. Here is a proper patch with a test case.

Changed 7 years ago by blc56

Attachment: delaunay_2_5d.2.patch added

comment:5 Changed 7 years ago by strk

Resolution: fixed
Status: newclosed

Committed in r3716, thanks !

comment:6 Changed 6 years ago by robe

Keywords: history added
Note: See TracTickets for help on using tickets.