[GRASS-dev] v.buffer

Hi all.
AFAIK, v.buffer has been deprecated, and replaced by v.buffer2. As a
result, the qgis-plugin-grass cannot use the command anymore. Of course
it is possible and esy to replace the command, but care should be taken
when linking to different grass versions. Wouldnì't it be easier/more
robust to either change the name of the new module back to v.buffer, or
to ln?
See: https://trac.osgeo.org/qgis/ticket/1629
All the best.
--
Paolo Cavallini: http://www.faunalia.it/pc

On Thu, Apr 23, 2009 at 8:50 AM, Paolo Cavallini <cavallini@faunalia.it> wrote:

Hi all.
AFAIK, v.buffer has been deprecated, and replaced by v.buffer2.

Yes.

As a result, the qgis-plugin-grass cannot use the command anymore.
Of course it is possible and esy to replace the command, but care should be taken
when linking to different grass versions. Wouldnì't it be easier/more
robust to either change the name of the new module back to v.buffer, or
to ln?

This is the case:

cat vector/v.buffer2/Makefile
MODULE_TOPDIR = ../..
PGM = v.buffer
...

The new version is called v.buffer as before (obviously).

See: https://trac.osgeo.org/qgis/ticket/1629

OK, I found the problem:
# GRASS 6.3, http://grass.osgeo.org/grass63/manuals/html63_user/v.buffer.html:
buffer=float
    Buffer distance in map units

# GRASS 6.4, http://grass.osgeo.org/grass64/manuals/html64_user/v.buffer.html:
distance=float
    Buffer distance along major axis in map units

This is bad.

Options:
- QGIS updates its toolbox XML file to rename the parameter
- GRASS 6.4+ renames back "distance" to "buffer"

Thanks for pointing out.
Markus

Markus Neteler ha scritto:

This is bad.

Options:
- QGIS updates its toolbox XML file to rename the parameter
- GRASS 6.4+ renames back "distance" to "buffer"

I would suggest the second, as it is simpler and keeps compatibility,
not only with QGIS.
Please let me know how you intend to proceed, so I can take the right
action.
All the best, and thanks.
--
Paolo Cavallini: http://www.faunalia.it/pc

On 23.04.2009 10:53, Markus Neteler wrote:

This is bad.

Options:
- QGIS updates its toolbox XML file to rename the parameter
- GRASS 6.4+ renames back "distance" to "buffer"

I think that the second option would be better. Users might have scripts
which use the old syntax. Aren't we supposed to be more or less
backwards compatible within a major version? However distance is a more
"proper" name for the parameter IMHO. So keep it as distance in 7 maybe?

--Wolf

--

<:3 )---- Wolf Bergenheim ----( 8:>

Hi,

2009/4/23 Wolf Bergenheim <wolf+grass@bergenheim.net>:

backwards compatible within a major version? However distance is a more
"proper" name for the parameter IMHO. So keep it as distance in 7 maybe?

+1 for 'distance' in GRASS7.

Martin

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