You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by Yexi Jiang <ye...@gmail.com> on 2013/12/03 00:43:16 UTC

Re: Review Request 13406: mahout-1265: add multilayer perceptron.

Suneel,

If this does not work, which location is the safe place to put the
temporary file?

Regards,
Yexi


2013/12/2 Suneel Marthi <su...@gmail.com>

> Yexi,
>
> The tests have to be redone in light of recent
> changes for m-1345.  We shouldn't be writing to /tmp
> anymore which is gonna fail the tests.
>
> More later.
>
> Sent from my iPhone
>
> On Dec 2, 2013, at 6:25 PM, "Yexi Jiang" <ye...@gmail.com> wrote:
>
>   This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13406/
>   Review request for mahout and Ted Dunning.
> By Yexi Jiang.
>
> *Updated Dec. 2, 2013, 11:25 p.m.*
> Changes
>
> I have updated the code according to the comments.
>
>   *Repository: * mahout
> Description
>
> mahout-1265: add multilayer perceptron. For details, please refer to https://issues.apache.org/jira/browse/MAHOUT-1265.
>
>   Testing
>
> Please see the corresponding test cases
>
>   Diffs (updated)
>
>    -
>    https://svn.apache.org/repos/asf/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/mlp/MultilayerPerceptron.java
>    (PRE-CREATION)
>    -
>    https://svn.apache.org/repos/asf/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/mlp/NeuralNetwork.java
>    (PRE-CREATION)
>    -
>    https://svn.apache.org/repos/asf/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/mlp/NeuralNetworkFunctions.java
>    (PRE-CREATION)
>    -
>    https://svn.apache.org/repos/asf/mahout/trunk/core/src/test/java/org/apache/mahout/classifier/mlp/TestMultilayerPerceptron.java
>    (PRE-CREATION)
>    -
>    https://svn.apache.org/repos/asf/mahout/trunk/core/src/test/java/org/apache/mahout/classifier/mlp/TestNeuralNetwork.java
>    (PRE-CREATION)
>
> View Diff <https://reviews.apache.org/r/13406/diff/>
>
>


-- 
------
Yexi Jiang,
ECS 251,  yjian004@cs.fiu.edu
School of Computer and Information Science,
Florida International University
Homepage: http://users.cis.fiu.edu/~yjian004/

Re: Review Request 13406: mahout-1265: add multilayer perceptron.

Posted by Andrew Musselman <an...@gmail.com>.
MahoutTestCase.getTestTempFilePath("example/path") should work according to
what I've seen.


On Mon, Dec 2, 2013 at 3:43 PM, Yexi Jiang <ye...@gmail.com> wrote:

> Suneel,
>
> If this does not work, which location is the safe place to put the
> temporary file?
>
> Regards,
> Yexi
>
>
> 2013/12/2 Suneel Marthi <su...@gmail.com>
>
> > Yexi,
> >
> > The tests have to be redone in light of recent
> > changes for m-1345.  We shouldn't be writing to /tmp
> > anymore which is gonna fail the tests.
> >
> > More later.
> >
> > Sent from my iPhone
> >
> > On Dec 2, 2013, at 6:25 PM, "Yexi Jiang" <ye...@gmail.com> wrote:
> >
> >   This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/13406/
> >   Review request for mahout and Ted Dunning.
> > By Yexi Jiang.
> >
> > *Updated Dec. 2, 2013, 11:25 p.m.*
> > Changes
> >
> > I have updated the code according to the comments.
> >
> >   *Repository: * mahout
> > Description
> >
> > mahout-1265: add multilayer perceptron. For details, please refer to
> https://issues.apache.org/jira/browse/MAHOUT-1265.
> >
> >   Testing
> >
> > Please see the corresponding test cases
> >
> >   Diffs (updated)
> >
> >    -
> >
> https://svn.apache.org/repos/asf/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/mlp/MultilayerPerceptron.java
> >    (PRE-CREATION)
> >    -
> >
> https://svn.apache.org/repos/asf/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/mlp/NeuralNetwork.java
> >    (PRE-CREATION)
> >    -
> >
> https://svn.apache.org/repos/asf/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/mlp/NeuralNetworkFunctions.java
> >    (PRE-CREATION)
> >    -
> >
> https://svn.apache.org/repos/asf/mahout/trunk/core/src/test/java/org/apache/mahout/classifier/mlp/TestMultilayerPerceptron.java
> >    (PRE-CREATION)
> >    -
> >
> https://svn.apache.org/repos/asf/mahout/trunk/core/src/test/java/org/apache/mahout/classifier/mlp/TestNeuralNetwork.java
> >    (PRE-CREATION)
> >
> > View Diff <https://reviews.apache.org/r/13406/diff/>
> >
> >
>
>
> --
> ------
> Yexi Jiang,
> ECS 251,  yjian004@cs.fiu.edu
> School of Computer and Information Science,
> Florida International University
> Homepage: http://users.cis.fiu.edu/~yjian004/
>