Or that is meant to be behavior of iterateAll() only? which is true for all iterators except SparseMatrix it seems.
On Tue, Jan 13, 2015 at 5:22 PM, Dmitriy Lyubimov <[email protected]> wrote: > Hi, > > I posted a question before about what it means to iterate over matrix, and > was given an answer it means going over matrix rows (0...m-1). > > That makes it, for exapmle, possible to create code like the following > > for ( (row, index) <- mxA.zipWithIndex ) row += something(index) > > however actual code in SparseMatrix iterates over hash of existing rows. > That breaks this execution for SparseMatrix only in two senses : > > (1) not all rows are visited; only those that have non-zeros. In > particular, if we created a matrix via like() then nothing is happening at > all; > > (2) rows are not visited in order of indices. > > I would like to fix (1) and (2) problems in this iterator but not sure if > this is important elsewhere. >
