[GRASS-dev] python port of r.denoise (testing required)

[Carlos Grohmann]

Thanks for testing it Anna.

I commited some changes: EPSG code is not required in a projected location and null values are repected.

Carlos



On 6 Dec 2016 02:16 -0200, Anna Petrášová <kratochanna at gmail.com>, wrote:
> Thank you for rewriting this, here is couple of brief observations.
> The epsg option is currently required in the code, although it
> shouldn't be necessary if you are in projected system. So it fails if
> you don't specify it. Also it should probably respect nulls in the
> input, now it results in -4e-06 instead of nulls.
>
> Best,
>
> Anna
>
>
> On Mon, Dec 5, 2016 at 5:16 PM, Carlos Grohmann
> <carlos.grohmann at gmail.com> wrote:
> > Ok,
> >
> > I used the html from the old module. Hadn’t realized that there was a
> > specific order for things in the html. Sorry.
> >
> > Thanks for the fix, Martin.
> >
> > Carlos
> >
> >
> >
> > On 5 Dec 2016 20:13 -0200, Markus Neteler <neteler at osgeo.org>, wrote:
> >
> > On Mon, Dec 5, 2016 at 9:23 PM, Martin Landa <landa.martin at gmail.com> wrote:
> >
> > Hi,
> >
> > 2016-12-05 21:14 GMT+01:00 Carlos Grohmann <carlos.grohmann at gmail.com>:
> >
> > The module itself was installed (and runs fine), but not the html help
> > page. What did I miss?
> >
> >
> > Also for addons, please follow the documentation guidelines:
> > https://trac.osgeo.org/grass/wiki/Submitting/Docs
> >
> > (I think that Martin already fixed most of it)
> >
> > Markus
> >
> >
> > _______________________________________________
> > grass-dev mailing list
> > grass-dev at lists.osgeo.org
> > http://lists.osgeo.org/mailman/listinfo/grass-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20161206/4467c9a9/attachment-0001.html>