Opened 12 years ago

Closed 12 years ago

Last modified 11 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: main
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 12 years ago.
delaunay_2_5d.2.patch​ (3.8 KB ) - added by blc56 12 years ago.

Download all attachments as: .zip

Change History (8)

by blc56, 12 years ago

Attachment: delaunay_2_5d.patch​ added

comment:1 by blc56, 12 years ago

Patch attached.

comment:2 by blc56, 12 years ago

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

comment:3 by strk, 12 years ago

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 by blc56, 12 years ago

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

by blc56, 12 years ago

Attachment: delaunay_2_5d.2.patch​ added

comment:5 by strk, 12 years ago

Resolution: → fixed
Status: new → closed

Committed in r3716, thanks !

comment:6 by robe, 11 years ago

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