[
https://issues.apache.org/jira/browse/MAHOUT-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adam Ilardi updated MAHOUT-1447:
--------------------------------
Description:
I added a test case for the YtY calculation code
I removed the indexes.quickSort() in the YtY calculation because I don't think
it's necessary and the test cases passed without it. The order shouldn't matter
since you're adding the scalers together. Correct me if i'm wrong.
In Factorization.java I added methods to access the iterator of item ids and
user ids directly. This saves memory when using classes like TopItems.java when
you don't have the DataModel class in memory as well.
> ImplicitFeedbackAlternatingLeastSquaresSolver tests and features
> ----------------------------------------------------------------
>
> Key: MAHOUT-1447
> URL: https://issues.apache.org/jira/browse/MAHOUT-1447
> Project: Mahout
> Issue Type: Improvement
> Components: Collaborative Filtering
> Affects Versions: 0.9
> Reporter: Adam Ilardi
> Assignee: Sebastian Schelter
> Priority: Minor
> Labels: newbie, patch, performance
> Fix For: 1.0
>
>
> I added a test case for the YtY calculation code
> I removed the indexes.quickSort() in the YtY calculation because I don't
> think it's necessary and the test cases passed without it. The order
> shouldn't matter since you're adding the scalers together. Correct me if i'm
> wrong.
> In Factorization.java I added methods to access the iterator of item ids and
> user ids directly. This saves memory when using classes like TopItems.java
> when you don't have the DataModel class in memory as well.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
