[jira] Commented: (MAHOUT-95) UserSimilarity-based NearestNNeighborhood

["Sean Owen (JIRA)" <ji...@apache.org>]

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

Sean Owen commented on MAHOUT-95:
---------------------------------

Looks OK to me. I agree it makes sense to maintain only 1 class if it encapsulates the other two. I changed the original today so you may want to look at the original again.

> UserSimilarity-based NearestNNeighborhood
> -----------------------------------------
>
>                 Key: MAHOUT-95
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-95
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Collaborative Filtering
>            Reporter: Otis Gospodnetic
>            Priority: Minor
>         Attachments: MAHOUT-95-diff-against-nearestN.txt, MAHOUT-95.patch
>
>
> A variation of NearestNUserNeighborhood.  This version adds the minSimilarity parameter, which is the primary factor for including/excluding other users from the target user's neighbourhood.  Additionally, the 'n' parameter was renamed to maxHoodSize and is used to optionally limit the size of the neighbourhood.
> The patch is for a brand new class, but we may really want just a single class (either keep this one and axe NearestNUserNeighborhood or add this functionality to NearestNUserNeighborhood), if this sounds good.
> I'll update the unit test and provide a patch for that if others think this can go in.
> Thoughts?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.