[
https://issues.apache.org/jira/browse/MAHOUT-1882?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15812862#comment-15812862
]
Dmitriy Lyubimov commented on MAHOUT-1882:
------------------------------------------
the fix is as easy as filtering 0s from non-default iterator (which i think is
done right for RASV) and i think it is important since it may induce spurious
non-deterministic behavior (e.g., switching results between 0 and NaN on
seemingly the same input of `x ::= { 1 / _ }` )
> SequentialAccessSparseVector inerateNonZeros is incorrect.
> ----------------------------------------------------------
>
> Key: MAHOUT-1882
> URL: https://issues.apache.org/jira/browse/MAHOUT-1882
> Project: Mahout
> Issue Type: Bug
> Components: Math
> Affects Versions: 0.12.2
> Reporter: Andrew Palumbo
> Assignee: Suneel Marthi
> Priority: Critical
> Fix For: 0.13.0
>
>
> In {{SequentialAccessSparseVector}} a bug is noted. When Cuonting Non-Zero
> elements {{NonDefaultIterator}} can, under certain circumstances give an
> incorrect iterator of size different from the actual non-zeroCounts.
> {code}
> @Override
> public Iterator<Element> iterateNonZero() {
> // TODO: this is a bug, since nonDefaultIterator doesn't hold to non-zero
> contract.
> return new NonDefaultIterator();
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
