[GRASS-dev] r.fillnulls in trunk broken when map units are not meter

GRASS GRASS developer
1

... because in the python script there is in line 120

    reg = grass.region()

but it should be the python equivalent to bash

eval `g.region -gm | grep res`

note the -m flag. This is necessary because the input buffer distance
for r.buffer must be in meters.

Markus M

2

Markus Metz wrote:

... because in the python script there is in line 120

reg = grass.region()

but it should be the python equivalent to bash

eval `g.region -gm | grep res`

note the -m flag. This is necessary because the input buffer distance
for r.buffer must be in meters.

Fixed in r42142, using r.grow instead of r.buffer, much simpler and
guaranteed 3 cells buffer, see original comments in code. Can someone
please review my python hack? Thanks.

BTW, r.buffer in trunk is broken for latln, wrong results and slow, in
grass64 results are fine and it is much faster (???).

Markus M

3

On Fri, May 7, 2010 at 1:43 PM, Markus Metz
<markus.metz.giswork@googlemail.com> wrote:
...

BTW, r.buffer in trunk is broken for latln, wrong results and slow, in
grass64 results are fine and it is much faster (???).

+1!

Markus