[
https://issues.apache.org/jira/browse/MAHOUT-896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13157223#comment-13157223
]
Anatoliy Kats commented on MAHOUT-896:
--------------------------------------
Patch submitted. The method name processUser is uninformative, since the word
'process' can refer to anything. The original names for the parameters are
also confusing: it is not clear that trainingUsers and testingUserPrefs refer
to the same kinds of items. The word 'Pref' is, ahem, preferable, since the
Preference class includes the User ID, Item ID, and Preference Value. So, I
renamed all the variables to use that term only.
What do you think?
> Improve readability of AbstractDifferenceRecommenderEvaluator class
> -------------------------------------------------------------------
>
> Key: MAHOUT-896
> URL: https://issues.apache.org/jira/browse/MAHOUT-896
> Project: Mahout
> Issue Type: Improvement
> Components: Collaborative Filtering
> Affects Versions: 0.6
> Reporter: Anatoliy Kats
> Assignee: Sean Owen
> Priority: Trivial
> Labels: newbie
> Attachments: MAHOUT-896.patch
>
> Original Estimate: 0.5h
> Remaining Estimate: 0.5h
>
> Change the variable and private method names so that they are internally
> consistent, and their purpose is more clear.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
