[jira] [Updated] (MAHOUT-1091) Bug in SequentialAccessSparseVector full iteration

2012-10-08 Thread Ted Dunning (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAHOUT-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ted Dunning updated MAHOUT-1091:


Attachment: 0001-MAHOUT-1091-Add-test-to-demonstrate-broken-iterator-.patch

Ooops.  Correct patch is here.

> Bug in SequentialAccessSparseVector full iteration
> --
>
> Key: MAHOUT-1091
> URL: https://issues.apache.org/jira/browse/MAHOUT-1091
> Project: Mahout
>  Issue Type: Bug
>  Components: Math
>Reporter: Ted Dunning
> Fix For: 0.8
>
> Attachments: 
> 0001-MAHOUT-1091-Add-test-to-demonstrate-broken-iterator-.patch, 
> 0001-MAHOUT-1091-Add-test-to-demonstrate-broken-iterator-.patch, 
> MAHOUT-1091.patch
>
>
> The iterator for the SequentialAccessSparseVector doesn't return any items 
> beyond the last non-zero.  This breaks some stuff pretty massively, but 
> hopefully doesn't break much user code since iterating through all elements 
> of a sparse vector is a relatively rare thing to do.

--
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


[jira] [Updated] (MAHOUT-1091) Bug in SequentialAccessSparseVector full iteration

2012-10-08 Thread Ted Dunning (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAHOUT-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ted Dunning updated MAHOUT-1091:


Attachment: 0001-MAHOUT-1091-Add-test-to-demonstrate-broken-iterator-.patch

Here is a patch relative to MAHOUT-1086 latest patch.  It includes a test that 
demonstrates the problem as well as code to repair it.

> Bug in SequentialAccessSparseVector full iteration
> --
>
> Key: MAHOUT-1091
> URL: https://issues.apache.org/jira/browse/MAHOUT-1091
> Project: Mahout
>  Issue Type: Bug
>  Components: Math
>Reporter: Ted Dunning
> Fix For: 0.8
>
> Attachments: 
> 0001-MAHOUT-1091-Add-test-to-demonstrate-broken-iterator-.patch, 
> MAHOUT-1091.patch
>
>
> The iterator for the SequentialAccessSparseVector doesn't return any items 
> beyond the last non-zero.  This breaks some stuff pretty massively, but 
> hopefully doesn't break much user code since iterating through all elements 
> of a sparse vector is a relatively rare thing to do.

--
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


[jira] [Updated] (MAHOUT-1091) Bug in SequentialAccessSparseVector full iteration

2012-10-08 Thread Paritosh Ranjan (JIRA)

 [ 
https://issues.apache.org/jira/browse/MAHOUT-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Paritosh Ranjan updated MAHOUT-1091:


Attachment: MAHOUT-1091.patch

Just tried out of curiosity. The patch contains the fix and a junit test 
demonstrating the fix. 

The math module's test cases pass after this fix. Have not checked the complete 
build yet. 

> Bug in SequentialAccessSparseVector full iteration
> --
>
> Key: MAHOUT-1091
> URL: https://issues.apache.org/jira/browse/MAHOUT-1091
> Project: Mahout
>  Issue Type: Bug
>  Components: Math
>Reporter: Ted Dunning
> Fix For: 0.8
>
> Attachments: MAHOUT-1091.patch
>
>
> The iterator for the SequentialAccessSparseVector doesn't return any items 
> beyond the last non-zero.  This breaks some stuff pretty massively, but 
> hopefully doesn't break much user code since iterating through all elements 
> of a sparse vector is a relatively rare thing to do.

--
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