Opened 7 years ago

Closed 6 years ago

#3426 closed defect (fixed)

Impossible to not compute shape statistics with i.segment.stats

Reported by: tgrippa Owned by: grass-dev@…
Priority: normal Milestone:
Component: Addons Version: unspecified
Keywords: Cc: mlennert
CPU: Unspecified Platform: Linux

Description

Hi,

It seems it is impossible to avoid computation of shape statistics using i.segment.stats.

When unticking all 'area measurements' in the GUI or when not sepcifying the 'area_measures' parameter in command line, the standards shape statistics are anyway computed.

Change History (5)

in reply to:  description comment:1 by mlennert, 7 years ago

Replying to tgrippa:

Hi,

It seems it is impossible to avoid computation of shape statistics using i.segment.stats.

When unticking all 'area measurements' in the GUI or when not sepcifying the 'area_measures' parameter in command line, the standards shape statistics are anyway computed.

The same is true for the raster measures. In the command line you can use raster_statistics= and/or area_measures= (i.e. no measures after the '=') to avoid calculation of the measures, but this is not possible in the GUI, AFAICT. I don't know if there is a possibility to provide default answers to a non-required parameter and allow to leave it empty in the GUI. The easiest solution would be to not provide any default answers.

If someone has another idea, I would love to hear it.

comment:2 by tgrippa, 7 years ago

Ok the solution with command line will make the deal for me since I'm not using a lot the GUI. Thanks

in reply to:  2 comment:3 by mlennert, 7 years ago

Replying to tgrippa:

Ok the solution with command line will make the deal for me since I'm not using a lot the GUI.

A generic solution that works everywhere would be nice. One possibility is a flag for "no shape statistics" (for raster statistics none are calculated if you do not provide a raster input map), but I want to avoid parameter/flag inflation if possible

comment:4 by mlennert, 7 years ago

After some more thinking I decided to go for the additional flag.

Implemented in r71602. Please test.

Last edited 7 years ago by mlennert (previous) (diff)

comment:5 by mlennert, 6 years ago

Resolution: fixed
Status: newclosed

Closing this as it has worked in our tests so far. Please reopen if necessary.

Note: See TracTickets for help on using tickets.