Opened 7 years ago

Closed 6 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 7 years ago.
Small subset from a LiDAR point cloud in ASCII format.
output_map.jpg (210.5 KB) - added by neteler 7 years ago.
Output map, partially wrong
v_voronoi.diff (14 bytes) - added by neteler 6 years ago.
patch revoked

Download all attachments as: .zip

Change History (10)

Changed 7 years ago by MichaelP

Attachment: temp_points.xyz.zip added

Small subset from a LiDAR point cloud in ASCII format.

Changed 7 years ago by neteler

Attachment: output_map.jpg added

Output map, partially wrong

comment:1 Changed 7 years ago by neteler

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

comment:2 in reply to:  1 ; Changed 7 years ago by marisn

Replying to neteler:

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

It looks too much like #957

comment:3 in reply to:  2 ; Changed 6 years ago by mmetz

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.

comment:4 in reply to:  3 Changed 6 years ago by neteler

Milestone: 6.4.36.4.4

Replying to mmetz:

Fixed in r57268.

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

Changed 6 years ago by neteler

Attachment: v_voronoi.diff added

patch revoked

comment:5 Changed 6 years ago by neteler

A suggestion is welcome how to backport r57268.

comment:6 in reply to:  5 Changed 6 years ago by mlennert

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 Changed 6 years ago by neteler

Resolution: fixed
Status: newclosed

Closed as fixed, reopen if needed.

Note: See TracTickets for help on using tickets.