Opened 12 years ago

Closed 11 years ago

#1682 closed defect (fixed)

incorrect output from v.voronoi

Reported by: MichaelP Owned by: grass-dev@…
Priority: major Milestone: 6.4.4
Component: Vector Version: 6.4.2
Keywords: v.voronoi Cc:
CPU: x86-64 Platform: Linux

Description

v.voronoi is generating nonsensical data. At times it appears to be generating correct output, but at other times, the data generated is corrupted. The attached data set is one such data set that triggers it to generate incorrect output.

The attached data set is a subset from a LiDAR point cloud in ascii format. Nad83 UTM 15N.

Attachments (3)

temp_points.xyz.zip​ (28.8 KB ) - added by MichaelP 12 years ago.
Small subset from a LiDAR point cloud in ASCII format.
output_map.jpg​ (210.5 KB ) - added by neteler 12 years ago.
Output map, partially wrong
v_voronoi.diff​ (14 bytes ) - added by neteler 11 years ago.
patch revoked

Download all attachments as: .zip

Change History (10)

by MichaelP, 12 years ago

Attachment: temp_points.xyz.zip​ added

Small subset from a LiDAR point cloud in ASCII format.

by neteler, 12 years ago

Attachment: output_map.jpg​ added

Output map, partially wrong

comment:1 by neteler, 12 years ago

Screenshot of output map which is partially wrong, added. Looks like a sort problem?

in reply to:  1 ; comment:2 by marisn, 12 years ago

Replying to neteler:

Screenshot of output map which is partially wrong, added. Looks like a sort problem?

It looks too much like #957

in reply to:  2 ; comment:3 by mmetz, 11 years ago

Replying to marisn:

Replying to neteler:

Screenshot of output map which is partially wrong, added. Looks like a sort problem?

It looks too much like #957

Fixed in r57268.

in reply to:  3 comment:4 by neteler, 11 years ago

Milestone: 6.4.3 → 6.4.4

Replying to mmetz:

Fixed in r57268.

Backport of r57268 attached (for G65) - to be tested.

by neteler, 11 years ago

Attachment: v_voronoi.diff​ added

patch revoked

comment:5 by neteler, 11 years ago

A suggestion is welcome how to backport r57268.

in reply to:  5 comment:6 by mlennert, 11 years ago

Replying to neteler:

A suggestion is welcome how to backport r57268.

AFAICT, backport has already happened in r57819 and r57820. Output looks fine to me, now.

Can this be closed ?

comment:7 by neteler, 11 years ago

Resolution: → fixed
Status: new → closed

Closed as fixed, reopen if needed.

Note: See TracTickets for help on using tickets.