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

[Carlos Grohmann]

I added a Makefile (adapted from r.roughness.vector), and also went and uploaded to SVN.

I’m not used to commiting to SVN so this is what I did:

- checked out the add-ons with svn
- created r.denoise dir
- added the files
- changed description.html to r.denoise.html
- remembered that I should’ve run module_svn_propset.sh, so I did this
- commited the changes

Then I tried to install it with g.extension but it didn’t worked:

g.extension r.denoise
Fetching <r.denoise> from GRASS GIS Addons repository (be patient)...
Compiling...
/var/folders/y9/mnj7kbf10pl0zfs74d03sz940000gn/T/grass7-guano-54185/tmptOIYEY/r.denoise/docs/html/r.denoise.html:75:0: Error (IndexError('pop from empty list',)): <em><b>r.denoise</em></b> requires that <em>mdenoise</em>, the executable version of Sun et al.'s (2007) denoising algorithm, is available on the $PATH.  <em>mdenoise</em> can be compiled and installed as follows:

make: *** [/var/folders/y9/mnj7kbf10pl0zfs74d03sz940000gn/T/grass7-guano-54185/tmptOIYEY/r.denoise/docs/man/man1/r.denoise.1] Error 1
ERROR: Compilation failed, sorry. Please check above error messages.
GRASS 7.2.0svn (latlong):~ >



The module itself  was installed (and runs fine), but not the html help page. What did I miss?

thanks





Carlos

--
Prof. Carlos Henrique Grohmann
Institute of Energy and Environment - Univ. of São Paulo, Brazil
- Digital Terrain Analysis | GIS | Remote Sensing -

carlosgrohmann.com
http://orcid.org/0000-0001-5073-5572
________________
Can’t stop the signal.

On 5 Dec 2016 14:42 -0200, Markus Neteler <neteler at osgeo.org>, wrote:
> On Fri, Dec 2, 2016 at 4:22 PM, Carlos Grohmann
> <carlos.grohmann at gmail.com> wrote:
> > Hello all.
> >
> > I just finished porting r.denoise to python, but before uploading it to the
> > add-ons, I would welcome any testing.
> >
> > The module is available here:
> > https://github.com/CarlosGrohmann/grass_gis/tree/master/r_denoise_py
>
> Please add a Makefile as well.
> In addition, you may consider to add the download and compilation of
> the needed mdenoise to the Makefile.
>
> Markus
>
> --
> Markus Neteler
> http://www.mundialis.de - free data with free software
> http://grass.osgeo.org
> http://courses.neteler.org/blog
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20161205/fb812091/attachment.html>