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

ASF GitHub Bot commented on MAHOUT-1630:
----------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/mahout/pull/115


> Incorrect SparseColumnMatrix.numSlices() causes IndexException in toString()
> ----------------------------------------------------------------------------
>
>                 Key: MAHOUT-1630
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1630
>             Project: Mahout
>          Issue Type: Bug
>          Components: Math
>    Affects Versions: 0.9
>            Reporter: Oleg Nitz
>            Assignee: Suneel Marthi
>              Labels: legacy, math, scala
>             Fix For: 0.10.0
>
>
> SparseColumnMatrix overrides the numSlices() method incorrectly: it returns 
> numCols() instead of numRows(). 
> As a result, AbstractMaxtrix.toString() for wide matrices throws an exception.
> For example, this code:
> {quote}
>     Matrix matrix = new SparseColumnMatrix(1, 2);
>     matrix.toString();
> {quote}
> causes
> {quote}
> org.apache.mahout.math.IndexException: Index 1 is outside allowable range of 
> [0,1)
>         at 
> org.apache.mahout.math.MatrixVectorView.<init>(MatrixVectorView.java:42)
>         at 
> org.apache.mahout.math.AbstractMatrix.viewRow(AbstractMatrix.java:290)
>         at 
> org.apache.mahout.math.AbstractMatrix$1.computeNext(AbstractMatrix.java:68)
>         at 
> org.apache.mahout.math.AbstractMatrix$1.computeNext(AbstractMatrix.java:59)
>         at 
> com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:143)
>         at 
> com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:138)
>         at 
> org.apache.mahout.math.AbstractMatrix.toString(AbstractMatrix.java:787)
> {quote}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to