i.segment.hierarchical: the outputs_prefix option should not require formatting string

[mlennert]

In i.segment.hierarchical, the outputs_prefix is defined as

outputs_prefix=string
    Name for output raster maps from segment
    Default: seg__%.2f

The way this parameter is used in the code, it requires a formatting string as present in the default. However, this is not clearly documented (except implicitly through the default value), and IMHO shouldn't be required. The formatting information should be added in the code, not by the user as is usual custom in GRASS modules.

I'll attach a patch shows a possible solution, but I'll wait for Pietro to comment on that before committing.

[zarch]

Replying to mlennert:

In i.segment.hierarchical, the outputs_prefix is defined as

outputs_prefix=string
    Name for output raster maps from segment
    Default: seg__%.2f

The way this parameter is used in the code, it requires a formatting string as present in the default. However, this is not clearly documented (except implicitly through the default value), and IMHO shouldn't be required. The formatting information should be added in the code, not by the user as is usual custom in GRASS modules.

Formatting strings are much more flexible but I agree that it is unusual, so please apply your changes.

[martinl]

Still relevant?

[martinl]

Remove Milestone from Addons bugreports.