Ticket #570 (closed defect: fixed)

Opened 11 months ago

Last modified 8 months ago

Add Z support in delaunay triangulation

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

Description

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

Attachments

delaunay_2_5d.patch Download (1.0 KB) - added by blc56 10 months ago.
delaunay_2_5d.2.patch Download (3.8 KB) - added by blc56 9 months ago.

Change History

Changed 10 months ago by blc56

Changed 10 months ago by blc56

Patch attached.

Changed 10 months ago by blc56

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

Changed 10 months 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 !

Changed 9 months ago by blc56

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

Changed 9 months ago by blc56

Changed 8 months ago by strk

  • status changed from new to closed
  • resolution set to fixed

Committed in r3716, thanks !

Note: See TracTickets for help on using tickets.