Hey Sean,
On Thu, Apr 15, 2010 at 7:16 AM, Sean Owen wrote:
> Along the way to a patch for MAHOUT-379, I'm having some trouble
> figuring out SequentialAccessSparseVector.DenseVector. I think it can
> be simplified, but unless I'm misunderstanding there are several bugs
> here. I'd like to find
Dmitriy raises some very good points here.
The actual implementation will need to address the small size block
problem. My guess is that the right way to do this is to touch the input
classes to make sure that the minimum split size is configurable.
-- Forwarded message --
From:
[
https://issues.apache.org/jira/browse/MAHOUT-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12857329#action_12857329
]
Danny Leshem edited comment on MAHOUT-369 at 4/15/10 10:21 AM:
--
[
https://issues.apache.org/jira/browse/MAHOUT-369?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Danny Leshem updated MAHOUT-369:
Attachment: MAHOUT-369.patch
Attached is a simple patch that fixed the two issues raised.
The righ
Along the way to a patch for MAHOUT-379, I'm having some trouble
figuring out SequentialAccessSparseVector.DenseVector. I think it can
be simplified, but unless I'm misunderstanding there are several bugs
here. I'd like to find my mistake or else simplify/fix this along the
way.
get() uses offset
