Hi,
I have completed the most of issues noted at [1] (todos for GRASS
6.4.3). Only one open issue remained - `v.vect.stats`. First of all
the module should be included in `devbr6`. I don't know the status of
this wish, so leaving open...
Martin
[1] http://trac.osgeo.org/grass/wiki/Grass6Planning#GRASS6.4.3
--
Martin Landa <landa.martin gmail.com> * http://geo.fsv.cvut.cz/~landa
On Mon, Mar 5, 2012 at 10:45 AM, Martin Landa <landa.martin@gmail.com> wrote:
Hi,
I have completed the most of issues noted at [1] (todos for GRASS
6.4.3).
Thanks for that!
Only one open issue remained - `v.vect.stats`. First of all
the module should be included in `devbr6`. I don't know the status of
this wish, so leaving open...
I observe some subtle differences between
grass70/vector/v.vect.stats/main.c
grass-addons/grass6/vector/v.vect.stats/main.c
which should be fixed, esp:
@@ -107,8 +107,8 @@
dbDriver *Pdriver, *Adriver;
char buf[2000];
int update_ok, update_err;
- struct ilist *List; <---
- BOUND_BOX box;
+ struct boxlist *List; <---
+ struct bound_box box;
and
@@ -416,7 +418,7 @@
Points = Vect_new_line_struct();
ACats = Vect_new_cats_struct();
PCats = Vect_new_cats_struct();
- List = Vect_new_list(); <---
+ List = Vect_new_boxlist(0); <---
No idea, leaving this to the author.
Markus
Hi,
2012/3/5 Markus Neteler <neteler@osgeo.org>:
- struct ilist *List; <---
- BOUND_BOX box;
+ struct boxlist *List; <---
+ struct bound_box box;
`boxlist` and `bound_box` have been introduced in GRASS 7.
struct ilist *List;
BOUND_BOX box;
is valid for GRASS 6.
--
Martin Landa <landa.martin gmail.com> * http://geo.fsv.cvut.cz/~landa
On 3/5/12, Martin Landa <landa.martin@gmail.com> wrote:
Hi,
2012/3/5 Markus Neteler <neteler@osgeo.org>:
- struct ilist *List; <---
- BOUND_BOX box;
+ struct boxlist *List; <---
+ struct bound_box box;`boxlist` and `bound_box` have been introduced in GRASS 7.
struct ilist *List;
BOUND_BOX box;is valid for GRASS 6.
All differences between the GRASS 6 and GRASS 7 version of
v.vect.stats are only related to API changes in GRASS 7.
The module has been tested for some time, by different users by now,
it is IMHO safe to include it in 6.4.3. The module does amongst other
things a PIP (points in polygon) count, a feature frequently
requested.
Markus M
(back to
http://trac.osgeo.org/grass/wiki/Grass6Planning#GRASS6.4.3
)
On Mon, Mar 5, 2012 at 5:36 PM, Markus Metz
<markus.metz.giswork@googlemail.com> wrote:
On 3/5/12, Martin Landa <landa.martin@gmail.com> wrote:
2012/3/5 Markus Neteler <neteler@osgeo.org>:
...
All differences between the GRASS 6 and GRASS 7 version of
v.vect.stats are only related to API changes in GRASS 7.The module has been tested for some time, by different users by now,
it is IMHO safe to include it in 6.4.3. The module does amongst other
things a PIP (points in polygon) count, a feature frequently
requested.
Unfortunately we omitted so far to move v.vect.stats from Addons into
core GRASS 6 - should this happen now? Any problems with that?
Markus
On Sat, Dec 29, 2012 at 11:22 PM, Markus Neteler <neteler@osgeo.org> wrote:
(back to
http://trac.osgeo.org/grass/wiki/Grass6Planning#GRASS6.4.3
)On Mon, Mar 5, 2012 at 5:36 PM, Markus Metz
<markus.metz.giswork@googlemail.com> wrote:On 3/5/12, Martin Landa <landa.martin@gmail.com> wrote:
2012/3/5 Markus Neteler <neteler@osgeo.org>:
...
All differences between the GRASS 6 and GRASS 7 version of
v.vect.stats are only related to API changes in GRASS 7.The module has been tested for some time, by different users by now,
it is IMHO safe to include it in 6.4.3. The module does amongst other
things a PIP (points in polygon) count, a feature frequently
requested.Unfortunately we omitted so far to move v.vect.stats from Addons into
core GRASS 6 - should this happen now? Any problems with that?
Done: v.vect.stats included from Addons (in r54472), it will be part
of GRASS 6.4.3 (and 6.5; already in GRASS 7).
Markus