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

[Veronica Andreo]

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 tested with the same map in
grass72 with no patch)... and if I set the flag -n, I get:

ERROR: r.out.xyz: Sorry, <n> is not a valid flag

Was that the expected result? or maybe i'm not using it properly... IIUC, I
would expect to get the nulls by default and do not get them if I set the
-n flag, is that right?

t.rast.out.xyz for the time series containing that map, gives an empty
file... as before.

Best,
Vero

2016-12-05 23:08 GMT+01:00 Markus Neteler <neteler at osgeo.org>:

> Hi Vero,
>
> On Wed, Nov 30, 2016 at 2:10 PM, Veronica Andreo <veroandreo at gmail.com>
> 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 difficult to change
> default
> > behaviour to export NULL values and keep then, the -n flag as in r.stats
> to
> > not export/report NULL values?
>
> attached a trivial (untested) patch make make r.out.xyz behave as the
> rest.
> Please check:
>
> patch -p0 < r.out.xyz.diff
>
> cheers
> Markus
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20161206/939845a0/attachment.html>