You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "Sean Owen (Commented) (JIRA)" <ji...@apache.org> on 2011/09/29 14:32:45 UTC
[jira] [Commented] (MAHOUT-695) Have LDADriver determine numWords
from input vectors
[ https://issues.apache.org/jira/browse/MAHOUT-695?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13117243#comment-13117243 ]
Sean Owen commented on MAHOUT-695:
----------------------------------
Hey Mat, looks like a good patch, and Jake likes it. I think it's OK to break callers in this regard. Is it probably OK to commit this, as far as you are concerned, or are there any other changes / concerns that should be addressed?
> Have LDADriver determine numWords from input vectors
> ----------------------------------------------------
>
> Key: MAHOUT-695
> URL: https://issues.apache.org/jira/browse/MAHOUT-695
> Project: Mahout
> Issue Type: Improvement
> Components: Clustering
> Affects Versions: 0.5
> Reporter: Mat Kelcey
> Assignee: Jake Mannix
> Priority: Minor
> Fix For: 0.6
>
> Attachments: mahout-695.patch, mahout-695.patch
>
>
> It bugged me that you needed to specify the number of words directly to the LDADriver
> eg ./bin/mahout lda \
> -i ./examples/bin/work/reuters-out-seqdir-sparse/tf-vectors \
> -o ./examples/bin/work/reuters-lda -k 20 -v 50000 -ow -x 20
> with this patch the ldadriver just checks a vector from the input to determine the size
> eg ./bin/mahout lda \
> -i ./examples/bin/work/reuters-out-seqdir-sparse/tf-vectors \
> -o ./examples/bin/work/reuters-lda -k 20 -ow -x 20
--
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