-
-
Notifications
You must be signed in to change notification settings - Fork 11.3k
Conversation
A formula for GibbsLDA++ (A C/C++ Implementation of Latent Dirichlet Allocation (LDA) using Gibbs Sampling for Parameter Estimation and Inference). http://gibbslda.sourceforge.net/
Hey guys, do I need to provide anything else to get this pull request merged in? |
require 'formula' | ||
|
||
class GibbsLda < Formula | ||
url 'http://downloads.sourceforge.net/project/gibbslda/GibbsLDA%2B%2B/0.2/GibbsLDA%2B%2B-0.2.tar.gz?r=&ts=1314109304&use_mirror=ignum' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't use the "use_mirror" part of Sourceforge URLs, that way we can let it detect the best mirror on its own.
Good point, thanks! Fixed in the last commit. |
Do you need anything else, other than the above commit? Please let me know and I'll help if I can. |
Looks good, though I do wonder about the formula name: should it be gibbslda++ rather than gibbs-lda? How is it usually referred to? The website consistently calls it "gibbslda++". |
Good point Jack. I've gone with "gibbslda" as I wasn't sure how/if to use plusses in the formula/file name. I'm happy to change it to "gibbslda++" though if you know how to do that. |
A formula for GibbsLDA++ (A C/C++ Implementation of Latent Dirichlet Allocation (LDA) using Gibbs Sampling for Parameter Estimation and Inference). http://gibbslda.sourceforge.net/ Closes Homebrew#7164. Signed-off-by: Adam Vandenberg <[email protected]>
A formula for GibbsLDA++ (A C/C++ Implementation of Latent Dirichlet Allocation (LDA) using Gibbs Sampling for Parameter Estimation and Inference). http://gibbslda.sourceforge.net/ Closes Homebrew#7164. Signed-off-by: Adam Vandenberg <[email protected]>
A formula for GibbsLDA++ (A C/C++ Implementation of Latent Dirichlet Allocation (LDA) using Gibbs Sampling for Parameter Estimation and Inference). http://gibbslda.sourceforge.net/