[GRASS-dev] [bug #4451] (grass) New options for g.region

GRASS GIS GRASS GIS developer
1

Hi,

can someone please review this g.region patch proposal?

https://intevation.de/rt/webrt?serial_num=4451

thanks
Markus

-------------------------------------------- Managed by Request Tracker

2

On Sunday 03 December 2006 13:00, Markus Neteler via RT wrote:

Hi,

can someone please review this g.region patch proposal?

https://intevation.de/rt/webrt?serial_num=4451

thanks
Markus

-------------------------------------------- Managed by Request Tracker

_______________________________________________
grass-dev mailing list
grass-dev@grass.itc.it
http://grass.itc.it/mailman/listinfo/grass-dev

While I cannot vouch for the code aspect of this, the new options would be
quite useful for my operations.

Thanks Markus and others!

--
Dylan Beaudette
Soils and Biogeochemistry Graduate Group
University of California at Davis
530.754.7341

3

Markus Neteler via RT wrote:

can someone please review this g.region patch proposal?

https://intevation.de/rt/webrt?serial_num=4451

I don't see any point in the size= option; I wouldn't expect it to be
used much, and you can always just pass the same value to rows= and
cols=.

The rest of it seems fine.

--
Glynn Clements <glynn@gclements.plus.com>

4

Hi,

I have updated the patch for the current CVS

+ reordering flags (Print)
+ capitalize messages/warnings/ferrors
- removing size parameter

Any objections?

Best, Martin

2006/12/4, Glynn Clements <glynn@gclements.plus.com>:

Markus Neteler via RT wrote:

> can someone please review this g.region patch proposal?
>
> https://intevation.de/rt/webrt?serial_num=4451

I don't see any point in the size= option; I wouldn't expect it to be
used much, and you can always just pass the same value to rows= and
cols=.

The rest of it seems fine.

--
Glynn Clements <glynn@gclements.plus.com>

_______________________________________________
grass-dev mailing list
grass-dev@grass.itc.it
http://grass.itc.it/mailman/listinfo/grass-dev

--
Martin Landa <landa.martin@gmail.com> * http://gama.fsv.cvut.cz/~landa *

(attachments)

g_region_rc.diff.gz (3.49 KB)

5

On 12/3/06 2:00 PM, "Markus Neteler via RT" <grass-bugs@intevation.de>
wrote:

Hi,

can someone please review this g.region patch proposal?

https://intevation.de/rt/webrt?serial_num=4451

thanks
Markus

Looks good to me.

Michael

__________________________________________
Michael Barton, Professor of Anthropology
School of Human Evolution & Social Change
Center for Social Dynamics & Complexity
Arizona State University

phone: 480-965-6213
fax: 480-965-7671
www: http://www.public.asu.edu/~cmbarton

6

Hi,

2006/12/4, Martin Landa <landa.martin@gmail.com>:

I have updated the patch for the current CVS

+ reordering flags (Print)
+ capitalize messages/warnings/ferrors
- removing size parameter

Any objections?

ok, commited to CVS.

Martin

2006/12/4, Glynn Clements <glynn@gclements.plus.com>:
>
> Markus Neteler via RT wrote:
>
> > can someone please review this g.region patch proposal?
> >
> > https://intevation.de/rt/webrt?serial_num=4451
>
> I don't see any point in the size= option; I wouldn't expect it to be
> used much, and you can always just pass the same value to rows= and
> cols=.
>
> The rest of it seems fine.
>
> --
> Glynn Clements <glynn@gclements.plus.com>
>
> _______________________________________________
> grass-dev mailing list
> grass-dev@grass.itc.it
> http://grass.itc.it/mailman/listinfo/grass-dev
>

--
Martin Landa <landa.martin@gmail.com> * http://gama.fsv.cvut.cz/~landa *

--
Martin Landa <landa.martin@gmail.com> * http://gama.fsv.cvut.cz/~landa *