[ 
https://issues.apache.org/jira/browse/MAHOUT-1642?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14577658#comment-14577658
 ] 

ASF GitHub Bot commented on MAHOUT-1642:
----------------------------------------

Github user dlyubimov commented on the pull request:

    https://github.com/apache/mahout/pull/134#issuecomment-110109114
  
    ok, without being in context of this algorithmically, this looks like a 
valid fix to me. I think I'd rely on the opinion on contributors and commit it 
even though it is to a taste recommender. If @srowen happens to come and say 
yay or nay, it'd be great.


> Iterator class within SimilarItems class always misses the first element
> ------------------------------------------------------------------------
>
>                 Key: MAHOUT-1642
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1642
>             Project: Mahout
>          Issue Type: Bug
>          Components: Collaborative Filtering
>            Reporter: Guohua Hao
>              Labels: legacy
>
> In the next() function of SimilarItemsIterator class within SimilarItems 
> class, variable 'index' is incremented before returning the actual element at 
> that position, therefore the first element when iterating will always be 
> missed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to