Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#1767 closed enhancement (fixed)

v.univar: add sum

Reported by: mlennert Owned by: grass-dev@…
Priority: normal Milestone: 6.4.3
Component: Vector Version: unspecified
Keywords: v.univar Cc:
CPU: Unspecified Platform: Unspecified

Description

I was surprised to discover that v.univar does not print out the sum of the variable asked for. The attached very simple patch (against 64_release) does that. Was there any reason I should know about why the sum was not included ?

And: is it acceptable to add the sum to grass6, or is this an unacceptable change in behaviour of a module and should only be added to grass7 ?

Moritz

Attachments (1)

v.univar.sum.diff (812 bytes) - added by mlennert 7 years ago.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by mlennert

Attachment: v.univar.sum.diff added

comment:1 Changed 7 years ago by neteler

Resolution: fixed
Status: newclosed

Since it is an "easy" patch, added to G6.4-G7 in r53533, r53534, 53535.

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

Replying to neteler:

Since it is an "easy" patch, added to G6.4-G7 in r53533, r53534, 53535.

Thanks. I just hope that this will not perturb any scripts that depend on v.univar output being in a specific order...

Moritz

comment:3 Changed 7 years ago by neteler

In the first place eval and -g flag should be used. Then I don't see any v.univar usage the official scripts not in the C code. Should be fine.

Note: See TracTickets for help on using tickets.