[GRASS-dev] G7: t.* consistency changes for parameters and flags

Hi,

I have taken liberty to streamline to some extent the parameter and
flag of the temporal t.+ modules for consistency (trunk r60608; relbr
r60609; more changes may be needed). The changes include

- use macros where possible (some new may yet be defined in the parser);
- standard field separator for text output;
- make messages identical as needed (potentially less confusion, less
work for translation);
- split long messages into label/description to avoid overly large GUI windows.

The following issues I came across:

1) Suboptimal flag name (-h versus --h) [several t.* modules]:

currently:
  -h Print the column names as first row
--h Print usage summary

IMHO better invert this to -c behaviour as in v.db.select:
  -c Do not include column names in output

unless there is a strong need to NOT put the column names by default
(which I think would be helpful for the user).

2) What does the -s flag do [several t.* modules]:

#% description: Activate spatial topology

perhaps change to
#% description: Check spatial topology

?

3) Wish:
The t.list* modules should have a new "output=" parameter like that
recently added to g.mlist. This will help e.g Windows users who cannot
easily use backticks.

Markus

On Fri, May 30, 2014 at 3:44 PM, Markus Neteler <neteler@osgeo.org> wrote:

I have taken liberty to streamline to some extent the parameter and
flag of the temporal t.+ modules for consistency (trunk r60608; relbr
r60609; more changes may be needed). The changes include

...

- standard field separator for text output;

I just see that a non-standard separator is hardcoded:

lib/python/temporal/list_stds.py: separator = "\t"

I'd suggest to change that to '|' as everywhere.
Also, the standard words "comma" etc of G_OPT_F_SEP are not yet accepted.

Markus

On Fri, May 30, 2014 at 4:43 PM, Markus Neteler <neteler@osgeo.org> wrote:

On Fri, May 30, 2014 at 3:44 PM, Markus Neteler <neteler@osgeo.org> wrote:

I have taken liberty to streamline to some extent the parameter and
flag of the temporal t.+ modules for consistency (trunk r60608; relbr
r60609; more changes may be needed). The changes include

...

- standard field separator for text output;

I just see that a non-standard separator is hardcoded:

I have now added support for the new grass.separator() function
developed by Huidae.

Markus