Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-05 Thread Vaclav Petras
On Fri, May 5, 2017 at 3:43 AM, Moritz Lennert wrote: > Markus is probably right: working with an intermediate file solves all > this and I don't think it would imply a serious speed decrease. > I have tested r.vect.stats (with pipe) and compared it with pipe in

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-05 Thread Martin Landa
Hi, 2017-05-05 15:23 GMT+02:00 Vaclav Petras : > Before it gave expected result for points with Z, e.g. in full NC loc: > > r.vect.stats in=elev_lid792_bepts out=ground > > Now it gives error (likely due to column=options['column']), but more right, it needs to be fixed

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-05 Thread Vaclav Petras
On Fri, May 5, 2017 at 3:55 AM, Martin Landa wrote: > BTW, meanwhile I changed in r71022 default > method from 'mean' (module was calculating mean value of categories) > to 'n' (number of points in cell). > Before it gave expected result for points with Z, e.g. in full

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-05 Thread Martin Landa
Hi, 2017-05-05 9:43 GMT+02:00 Moritz Lennert : > Markus is probably right: working with an intermediate file solves all > this and I don't think it would imply a serious speed decrease. I would tend to agree. BTW, meanwhile I changed in r71022 default method from

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-05 Thread Markus Metz
On Fri, May 5, 2017 at 5:07 AM, Vaclav Petras wrote: > > On Wed, May 3, 2017 at 5:34 AM, Moritz Lennert < mlenn...@club.worldonline.be> wrote: > > > > On 02/05/17 15:53, Vaclav Petras wrote: > >> I'm using pipe_command() which is just convenience function setting > >>

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-04 Thread Vaclav Petras
On Wed, May 3, 2017 at 5:34 AM, Moritz Lennert wrote: > > On 02/05/17 15:53, Vaclav Petras wrote: >> I'm using pipe_command() which is just convenience function setting >> stdout=PIPE. Similarly feed_command() is just setting stdin=PIPE which >> I'm not using because

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-03 Thread Moritz Lennert
On 02/05/17 15:53, Vaclav Petras wrote: On Tue, May 2, 2017 at 3:24 AM, Moritz Lennert > wrote: But I'm not against a wrapper. I added a prototype with limited functionality to addons [1]. However, I'm not sure

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-02 Thread Vaclav Petras
On Tue, May 2, 2017 at 3:24 AM, Moritz Lennert wrote: > But I'm not against a wrapper. I added a prototype with limited >> functionality to addons [1]. However, I'm not sure how to account for >> large data - that's what discouraged me from creating a wrapper. The

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-02 Thread Moritz Lennert
On 02/05/17 03:03, Vaclav Petras wrote: [Switching from grass-user to grass-dev.] On Mon, May 1, 2017 at 5:11 PM, Markus Metz > wrote: > > Is it different from the binning code in r.in.xyz ? > >

Re: [GRASS-dev] [GRASS-user] r.to.vect stats

2017-05-01 Thread Vaclav Petras
[Switching from grass-user to grass-dev.] On Mon, May 1, 2017 at 5:11 PM, Markus Metz wrote: > > > Is it different from the binning code in r.in.xyz ? > > > > > > Not really. r.in.lidar was based on r.in.xyz, now they are different > but of course the idea is to