[MAHOUT-1729] Straighten out behavior of Matrix.iterator() and iterateNonEmpty()

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.10.2
• Assignee: Dmitriy Lyubimov
• Reporter: Dmitriy Lyubimov
• Created: 02/Jun/15 23:46
• Updated: 07/Aug/15 15:56
• Resolved: 17/Jun/15 18:59

Description

SparseMatrix in particular does iterateNonEmpty() instead of iterating over all rows in default iterator. This creates contract inconsistency leading to computational errors.

so... this fixes to guarantee that

for (row <- mxA)

visits ALL rows in the mxA (even if they are totally empty).

If it is ok not to process completely 0ed rows, then the following form should be used:

for (row <- mxA.iterateNonEmpty) ...

PR https://github.com/apache/mahout/pull/135