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

Robin Anil commented on MAHOUT-157:
-----------------------------------

Respose  one by one. 

1) Didnt realise it was kept in test.  I will move it shortly.

2) I made a couple of changes in the class Parameters which I frequently use to 
serialize and deserialize data in and out of map/reduce jobs both in Bayes and 
FPgrowth. Since the incremental change applies only to PFPG, it would be great 
if MAHOUT-124 gets committed soon after I fix the changes suggested there.

3) Tests and Docs (I am already on it)

4) i dont need to keep any particular state in PFPgrowth. I can change the 
construct to PFPGrowth.executeJob(params).

5) Re: "Is it possible to break test" Currently this patch has only 
FPGrowth(sequential) test. PFPGrowth tests will be put up soon

6) Fixed

7) TreeNode is tha basic unit of the FP-Tree and Conditional FPTree. it doesn't 
need any method. I didnt use getNext(),  getParent() etc to get members to 
speed up things i.e to remove the overhead of a very frequent function call






> Frequent Pattern Mining using Parallel FP-Growth
> ------------------------------------------------
>
>                 Key: MAHOUT-157
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-157
>             Project: Mahout
>          Issue Type: New Feature
>    Affects Versions: 0.2
>            Reporter: Robin Anil
>             Fix For: 0.2
>
>         Attachments: MAHOUT-157-August-6.patch, 
> MAHOUT-157-Combinations-BSD-License.patch, 
> MAHOUT-157-Combinations-BSD-License.patch, 
> MAHOUT-157-inProgress-August-5.patch
>
>
> Implement: http://infolab.stanford.edu/~echang/recsys08-69.pdf

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

Reply via email to