[ 
https://issues.apache.org/jira/browse/MAHOUT-259?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12801660#action_12801660
 ] 

Benson Margulies commented on MAHOUT-259:
-----------------------------------------

In some ways, this is clearer than the patch ...

{noformat}
D       math/src/main/java/org/apache/mahout/math/matrix/ObjectMatrix1D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SelectedSparseObjectMatrix1D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SelectedSparseObjectMatrix2D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SelectedSparseObjectMatrix3D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SparseObjectMatrix1D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/DenseObjectMatrix1D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SparseObjectMatrix2D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/DenseObjectMatrix2D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SparseObjectMatrix3D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/DenseObjectMatrix3D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SelectedDenseObjectMatrix1D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SelectedDenseObjectMatrix2D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/impl/SelectedDenseObjectMatrix3D.java
D       math/src/main/java/org/apache/mahout/math/matrix/ObjectMatrix2D.java
D       math/src/main/java/org/apache/mahout/math/matrix/ObjectMatrix3D.java
D       
math/src/main/java/org/apache/mahout/math/matrix/ObjectMatrix1DProcedure.java
D       math/src/main/java/org/apache/mahout/math/matrix/objectalgo
D       
math/src/main/java/org/apache/mahout/math/matrix/objectalgo/Formatter.java
D       math/src/main/java/org/apache/mahout/math/matrix/objectalgo/Sorting.java
D       
math/src/main/java/org/apache/mahout/math/matrix/objectalgo/Partitioning.java
D       
math/src/main/java/org/apache/mahout/math/matrix/objectalgo/ObjectMatrix1DComparator.java
D       
math/src/main/java/org/apache/mahout/math/matrix/objectalgo/ObjectMatrix2DComparator.java
D       math/src/main/java/org/apache/mahout/math/matrix/objectalgo/package.html
D       
math/src/main/java/org/apache/mahout/math/matrix/ObjectMatrix2DProcedure.java
D       
math/src/main/java/org/apache/mahout/math/matrix/ObjectMatrix3DProcedure.java
{noformat}


> Remove all code for Object matrices
> -----------------------------------
>
>                 Key: MAHOUT-259
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-259
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Math
>    Affects Versions: 0.3
>            Reporter: Benson Margulies
>         Attachments: MAHOUT-259.patch
>
>
> No code in Mahout that does math, even code straight from Colt, uses the 
> Object matrix classes. They are something of a nightmare from a Generic type 
> standpoint. 
> So, 'When in doubt, cut it out.'

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to