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

[Anna Petrášová]

On Tue, Dec 6, 2016 at 7:34 AM, Carlos Grohmann
<carlos.grohmann at gmail.com> wrote:
> Thanks for testing it Anna.
>
> I commited some changes: EPSG code is not required in a projected location
> and null values are repected.

Great, thank you, I will test later.

Anna

>
> 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