Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Markus Neteler
On Dec 19, 2016 5:41 AM, wrote: > > Thanks for this – I have made some major changes to r.randomforest recently, including adding options for spatial cross-validation. This is great! > I will move the import of pandas to a lazy import because I’m only using pandas as

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Helmut Kudrnovsky
> I will move the import of pandas to a lazy import because AFAIU also the Scikit imports should be done by a lazy import. - best regards Helmut -- View this message in context: http://osgeo-org.1560.x6.nabble.com/r-randomforest-built-status-tp5300348p5300387.html Sent from the Grass -

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread dr.stevenpawley
. Is there a correct protocol for me to rename the script to something like r.scikit.learn, or r.ml? Steve From: Vaclav Petras Sent: Sunday, December 18, 2016 2:42 PM To: Helmut Kudrnovsky Cc: grass-dev@lists.osgeo.org Subject: Re: [GRASS-dev] r.randomforest built status On Sun, Dec 18, 2016 at 3:40 PM

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Vaclav Petras
On Sun, Dec 18, 2016 at 3:40 PM, Helmut Kudrnovsky wrote: > > > > Is there any "best practice" example available? > > > I've done it in this way : > > https://trac.osgeo.org/grass/browser/grass-addons/grass7/ >

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Helmut Kudrnovsky
Markus Neteler wrote > On Sun, Dec 18, 2016 at 9:17 PM, Helmut Kudrnovsky > hellik@ > wrote: >> Helmut Kudrnovsky wrote > ... >>> what about to do some kind of a lazy import and move panda and scikit to >>> def main()? >>> >>> It isn't needed for compiling only for running the python script.

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Markus Neteler
On Sun, Dec 18, 2016 at 9:17 PM, Helmut Kudrnovsky wrote: > Helmut Kudrnovsky wrote ... >> what about to do some kind of a lazy import and move panda and scikit to >> def main()? >> >> It isn't needed for compiling only for running the python script. >> >> AFAIK panda is available

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Helmut Kudrnovsky
Helmut Kudrnovsky wrote > > Markus Neteler wrote >> On Sun, Dec 18, 2016 at 7:52 PM, Paulo van Breugel >> >> p.vanbreugel@ >> wrote: >>> Hi Steven / devs >>> >>> The build status of r.randomforest shows it failed to build. From the >>> log >>> file this seems to be because Pandas is not

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Helmut Kudrnovsky
Markus Neteler wrote > On Sun, Dec 18, 2016 at 7:52 PM, Paulo van Breugel > > p.vanbreugel@ > wrote: >> Hi Steven / devs >> >> The build status of r.randomforest shows it failed to build. From the log >> file this seems to be because Pandas is not installed >> >> ... >> ERROR: Pandas not

Re: [GRASS-dev] r.randomforest built status

2016-12-18 Thread Markus Neteler
On Sun, Dec 18, 2016 at 7:52 PM, Paulo van Breugel wrote: > Hi Steven / devs > > The build status of r.randomforest shows it failed to build. From the log > file this seems to be because Pandas is not installed > > ... > ERROR: Pandas not installed > make: ***

[GRASS-dev] r.randomforest built status

2016-12-18 Thread Paulo van Breugel
Hi Steven / devs The build status of r.randomforest shows it failed to build. From the log file this seems to be because Pandas is not installed ... ERROR: Pandas not installed make: *** [r.randomforest.tmp.html] Error 1 rm r.randomforest.tmp.html ... This is also why, I guess, it is not on