Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hi, sorry for the delay but last days was really intense... On 7 December 2016 at 22:00, Markus Neteler wrote: > > I overlooked that t.rast.out.xyz is an addon that depends on the release > branch. > Ok, while I dislike it I have backported the new flag in r70034. > I

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On Wed, Dec 7, 2016 at 9:10 AM, Veronica Andreo wrote: > Hi, > > 2016-12-06 23:24 GMT+01:00 Martin Landa : >> >> Hi, >> >> 2016-12-06 23:06 GMT+01:00 Markus Neteler : >> > I have submitted a new -i flag to include no data values

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hi, 2016-12-06 23:24 GMT+01:00 Martin Landa : > Hi, > > 2016-12-06 23:06 GMT+01:00 Markus Neteler : > > I have submitted a new -i flag to include no data values (r70017). > > This can be used for t.rast.out.xyz and backported for 7.2.1. > IMHO, if we

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hi, 2016-12-06 23:06 GMT+01:00 Markus Neteler : > I have submitted a new -i flag to include no data values (r70017). > This can be used for t.rast.out.xyz and backported for 7.2.1. recorded at [1]. Does it mean that we can go ahead and release 7.2.0? Ma [1]

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On Tue, Dec 6, 2016 at 9:15 PM, Markus Neteler wrote: > On Tue, Dec 6, 2016 at 7:34 PM, Vaclav Petras wrote: >> On Tue, Dec 6, 2016 at 12:34 PM, Moritz Lennert > ... >>> Why not make the flag mean the contrary, i.e. include null values ? Then >>> it

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On Tue, Dec 6, 2016 at 7:34 PM, Vaclav Petras wrote: > On Tue, Dec 6, 2016 at 12:34 PM, Moritz Lennert ... >> Why not make the flag mean the contrary, i.e. include null values ? Then >> it wouldn't break anything... >> >> I'm pretty sure that most people exporting data via

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On Tue, Dec 6, 2016 at 12:34 PM, Moritz Lennert < mlenn...@club.worldonline.be> wrote: > Le 6 décembre 2016 17:53:48 GMT+01:00, Martin Landa < > landa.mar...@gmail.com> a écrit : > >Hi, > > > >2016-12-06 17:45 GMT+01:00 Markus Neteler : > >> Big question: add to 7.2.0 or not? I

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hi, 2016-12-06 17:45 GMT+01:00 Markus Neteler : > Big question: add to 7.2.0 or not? I am in favor. +1 Ma -- Martin Landa http://geo.fsv.cvut.cz/gwiki/Landa http://gismentors.cz/mentors/landa ___ grass-dev mailing list

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On Tue, Dec 6, 2016 at 12:38 PM, Veronica Andreo wrote: > I'm so sorry for the noise!! I forgot to run make :-P Anyway, I just did and > tested again... works as expected!!! Cool!! :)) > > Thank you, Markus, for the patch and Moritz for the follow up!! great, it was a quick

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

I'm so sorry for the noise!! I forgot to run make :-P Anyway, I just did and tested again... works as expected!!! Cool!! :)) Thank you, Markus, for the patch and Moritz for the follow up!! Cheers, Vero 2016-12-06 12:31 GMT+01:00 Moritz Lennert : > On 06/12/16

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On 06/12/16 10:53, Veronica Andreo wrote: Hello Markus Thanks for the patch :) I applied it in my grass trunk and just tested for a map (146880 cells) that contains no data (45757 cells). I get all the pixels with data (101123), but nothing for the pixels with no data. It seems to be the same

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hello Markus Thanks for the patch :) I applied it in my grass trunk and just tested for a map (146880 cells) that contains no data (45757 cells). I get all the pixels with data (101123), but nothing for the pixels with no data. It seems to be the same as before, no? (It is indeed, the same. I

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hi Vero, On Wed, Nov 30, 2016 at 2:10 PM, Veronica Andreo wrote: > Hello devs, > > AFAIU, to activate the NULL export in r.out.xyz is then a trivial change, > but we do not know which letter to use as flag because -n is already used to > indicate the opposite behaviour (?).

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On 30 November 2016 at 14:26, Markus Neteler wrote: > On Wed, Nov 30, 2016 at 2:10 PM, Veronica Andreo wrote: >> Hello devs, >> >> AFAIU, to activate the NULL export in r.out.xyz is then a trivial change, >> but we do not know which letter to use as flag

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On Wed, Nov 30, 2016 at 2:10 PM, Veronica Andreo wrote: > Hello devs, > > AFAIU, to activate the NULL export in r.out.xyz is then a trivial change, > but we do not know which letter to use as flag because -n is already used to > indicate the opposite behaviour (?). Is it too

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hello devs, AFAIU, to activate the NULL export in r.out.xyz is then a trivial change, but we do not know which letter to use as flag because -n is already used to indicate the opposite behaviour (?). Is it too difficult to change default behaviour to export NULL values and keep then, the -n flag

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On Tue, Nov 29, 2016 at 3:38 PM, Luca Delucchi wrote: ... > I propose to add a flag to r.out.xyz to permit to export also null > values, what do other devs think? Since it is a trivial change in r.out.xyz.py (conditionalize the r.stats' flag -n Do not report no data

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On 29 November 2016 at 15:28, Veronica Andreo wrote: > Ciao Lu, > Ciao, > Yes, of course I have NULL cells... cuac! > > So, IIUC, as t.rast.out.xyz is a wraper for r.out.xyz it won't export cells > with NULL values, and from what I learnt just by trial and error, if the >

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Ciao Lu, Yes, of course I have NULL cells... cuac! So, IIUC, as t.rast.out.xyz is a wraper for r.out.xyz it won't export cells with NULL values, and from what I learnt just by trial and error, if the time series happens to start with a NULL cell then one gets an empty output. Hence, it's only

Re: [GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

On 28 November 2016 at 14:53, Veronica Andreo wrote: > Hello devs, > Hi Vero, > I'm using the add-on t.rast.out.xyz to export my raster time series to a csv > file. Here's my region definition: > > GRASS 7.3.svn (latlong_wgs84):~ > g.region -p >

[GRASS-dev] t.rast.out.xyz not exporting all cells in the computational region

Hello devs, I'm using the add-on t.rast.out.xyz to export my raster time series to a csv file. Here's my region definition: GRASS 7.3.svn (latlong_wgs84):~ > g.region -p raster=A20133052013312.L3m_8D_CHL_chlor_a_4km_arg projection: 3 (Latitude-Longitude) zone: 0 datum: wgs84