[ 
https://issues.apache.org/jira/browse/HAMA-699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13807830#comment-13807830
 ] 

Martin Illecker commented on HAMA-699:
--------------------------------------

Yes, I would need VectorWritable and DenseDoubleVector in core for a test case. 
(cyclic dependency of core and ml)
But regarding to ChiaHung Lin, maybe we should reduce the patch a little bit? 
And move writables from o.a.h.c.io.w to o.a.h.c.io. 

I will create a new reduced patch including the following structure:
{code}
org.apache.hama.commons.io.DenseVectorWritable.java
org.apache.hama.commons.io.MatrixWritable.java
org.apache.hama.commons.io.SparseVectorWritable.java
org.apache.hama.commons.io.StringArrayWritable.java
org.apache.hama.commons.io.TextArrayWritable.java
org.apache.hama.commons.io.VectorWritable.java

org.apache.hama.commons.math.CrossEntropy.java
org.apache.hama.commons.math.DenseDoubleMatrix.java
org.apache.hama.commons.math.DenseDoubleVector.java
org.apache.hama.commons.math.DoubleDoubleFunction.java
org.apache.hama.commons.math.DoubleDoubleVectorFunction.java
org.apache.hama.commons.math.DoubleFunction.java
org.apache.hama.commons.math.DoubleMatrix.java
org.apache.hama.commons.math.DoubleVector.java
org.apache.hama.commons.math.DoubleVectorFunction.java
org.apache.hama.commons.math.Function.java
org.apache.hama.commons.math.FunctionFactory.java
org.apache.hama.commons.math.IdentityFunction.java
org.apache.hama.commons.math.Sigmoid.java
org.apache.hama.commons.math.SquaredError.java
org.apache.hama.commons.math.Tanh.java
org.apache.hama.commons.math.Tuple.java

org.apache.hama.commons.util.KeyValuePair.java
org.apache.hama.commons.util.TextPair.java

Test classes:
org.apache.hama.commons.math.TestDenseDoubleMatrix.java
org.apache.hama.commons.math.TestDenseDoubleVector.java
org.apache.hama.commons.math.TestFunctionFactory.java
{code}

Only org.apache.hama.ml.math, KeyValuePair, TextPair and writables will be 
moved to commons.

> Add commons module
> ------------------
>
>                 Key: HAMA-699
>                 URL: https://issues.apache.org/jira/browse/HAMA-699
>             Project: Hama
>          Issue Type: Improvement
>            Reporter: Edward J. Yoon
>             Fix For: 0.7.0
>
>         Attachments: commons-module.txt, HAMA-699.patch
>
>
> As we disscussed, I'd like to add commons module.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to