Opened 10 years ago

Closed 6 years ago

#2183 closed enhancement (fixed)

v.centerpoint: category values in ascii output should include original cat values

Reported by: mlennert Owned by: grass-dev@…
Priority: normal Milestone: 7.4.1
Component: Addons Version: svn-trunk
Keywords: v.centerpoint ascii output Cc:
CPU: Unspecified Platform: Unspecified

Description

When using v.centerpoint to print the centerpoint coordinates to stdout, cat values represent the type of point (7 = mean of area, 8 = median of area, etc). I would find it more useful if in the ascii output the cat value would be that of the original feature (just as normally foreseen for vector output of v.centerpoint) and that the centroid type was an additional column.

Attachments (1)

v_centerpoint_backtrace.txt (21.7 KB ) - added by mlennert 10 years ago.
backtrace for segfault

Download all attachments as: .zip

Change History (10)

in reply to:  description ; comment:1 by mmetz, 10 years ago

Replying to mlennert:

When using v.centerpoint to print the centerpoint coordinates to stdout, cat values represent the type of point (7 = mean of area, 8 = median of area, etc). I would find it more useful if in the ascii output the cat value would be that of the original feature (just as normally foreseen for vector output of v.centerpoint) and that the centroid type was an additional column.

Please try r58842. Note that center points for point clouds have only the type of center point as cat and not all categories of all original features.

in reply to:  1 comment:2 by mlennert, 10 years ago

Replying to mmetz:

Replying to mlennert:

When using v.centerpoint to print the centerpoint coordinates to stdout, cat values represent the type of point (7 = mean of area, 8 = median of area, etc). I would find it more useful if in the ascii output the cat value would be that of the original feature (just as normally foreseen for vector output of v.centerpoint) and that the centroid type was an additional column.

Please try r58842. Note that center points for point clouds have only the type of center point as cat and not all categories of all original features.

Thanks, it looks good, but now I get a segfault. I'll attach a full backtrace.

Moritz

by mlennert, 10 years ago

Attachment: v_centerpoint_backtrace.txt added

backtrace for segfault

comment:3 by mlennert, 10 years ago

FYI: I don't get that segfault when I send output to a new vector layer (i.e. when I use the output parameter).

comment:4 by martinl, 8 years ago

Milestone: 7.0.07.0.5

comment:5 by martinl, 8 years ago

Milestone: 7.0.57.3.0

comment:6 by martinl, 8 years ago

Milestone: 7.3.07.4.0

Milestone renamed

comment:7 by martinl, 6 years ago

What is the status of this issue?

comment:8 by neteler, 6 years ago

Milestone: 7.4.07.4.1

Ticket retargeted after milestone closed

in reply to:  7 comment:9 by mlennert, 6 years ago

Resolution: fixed
Status: newclosed

Replying to martinl:

What is the status of this issue?

I cannot reproduce the segfault, and the original wish has been implemented, so closing.

Note: See TracTickets for help on using tickets.