[ 
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

Reply via email to