[ https://issues.apache.org/jira/browse/MAHOUT-1193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13642752#comment-13642752 ]
Gokhan Capan commented on MAHOUT-1193: -------------------------------------- Sorry I missed that. I modified the SparseMatrix code to handle dense rows and I am happy with that. The code is not patch-quality, but I can implement a flexible extension to the current implementation if that is desired (I believe that might be a common use case). I personally liked the BlockSparseMatrix idea and its really flexible schema. I did a quick implementation to make it work with configurable block size, in a few days I can submit an additional diff to the reviewboard so we can discuss on code. One thing to consider, I suspect my version's CPU usage is kind of high. I believe both versions are valuable and important, they have their own benefits, particularly as an input to online learning algorithms. > We may want a BlockSparseMatrix > ------------------------------- > > Key: MAHOUT-1193 > URL: https://issues.apache.org/jira/browse/MAHOUT-1193 > Project: Mahout > Issue Type: Bug > Reporter: Ted Dunning > Attachments: MAHOUT-1193.patch > > > Here is an implementation. > Is it good enough to commit? > Is it useful? > Is it redundant? -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira