Hi Sebastian,
Thanks for have a look at my patch. I re-created review request on the
review board with a new patch.
On 5 May 2014 15:03, Sebastian Schelter (JIRA) <jira@apache.org> wrote:
>
> [
> https://issues.apache.org/jira/browse/MAHOUT-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13989284#comment-13989284]
>
> Sebastian Schelter commented on MAHOUT-1428:
> --------------------------------------------
>
> Dodi,
>
> I had a first look at your patch. There is one big issue with it: you
> modify the signature of recommend to add a parameter. Please add a new
> recommend method with the parameter instead, keep the old method around
> and make a default implementation that delegates to the new method with a
> default value of false for your new parameter. At the moment, you break all
> existing code that uses the Recommender interface, that's also why you have
> to edit some many files.
>
> Furthermore, please rename the new parameter to 'includeKnownItems', I
> think that is easier to understand than 'isUserItemInclude'.
>
> > Recommending already consumed items
> > -----------------------------------
> >
> > Key: MAHOUT-1428
> > URL: https://issues.apache.org/jira/browse/MAHOUT-1428
> > Project: Mahout
> > Issue Type: Bug
> > Components: Collaborative Filtering
> > Affects Versions: 1.0
> > Reporter: Mario Levitin
> > Labels: easyfix
> > Fix For: 1.0
> >
> > Attachments: MAHOUT-1428.patch
> >
> >
> > Mahout does not recommend items which are already consumed by the user.
> > For example,
> > In the getAllOtherItems method of GenericUserBasedRecommender class
> there is the following line
> > possibleItemIDs.removeAll(dataModel.getItemIDsFromUser(theUserID));
> > which removes user's items from the possibleItemIDs to prevent these
> items from being recommended to the user. This is ok for many
> recommendation cases but for many other cases it is not.
> > The Recommender classes (I mean all of them, NN-based and SVD-based as
> well as hadoop and non-hadoop versions) might have a parameter for this for
> excluding or not excluding user items in the returned recommendations.
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.2#6252)
>
--
Best Regards,
Dodi Amar Hakim
|