-
Notifications
You must be signed in to change notification settings - Fork 233
Support asymmetric Dirichlet prior #22
base: master
Are you sure you want to change the base?
Conversation
|
||
if(Config::asymmetric_prior) | ||
{ | ||
// Request topic-frequency-table |
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.
Only need to request the table if iter % num_alpha_iteration == 0?
Thanks for the great work!@hiyijian I'm really sorry for my late response. The implementation should be OK. I review the code and add some notes. I think it's OK to merge to the master. |
@hiyijian I'm trying to use asymmetric lda. count you tell me your QQ or Wechat? I have some questions. |
@lisendong My implemention is on the top of source codes provided by @feiga , which I think is used in Microsoft only. However, my implemention is incorrect, I think. Fell free to concact me via @hiyijian@qq.com |
@hiyijian I'm tring to use your asymmetric prior version of lightlda that based on microsoft's code,I am seeing that you said "my implemention is incorrect",is that meaning your code still had some bug? |
hi @tangzhenyu and @koustuvsinha. I said "this implemention is incorrect" since I have not seen any improvement compared by feiga's original symmetric LDA, Sometimes seems even worse :( . |
According to Wallach’s paper, asymmetric, hierarchical Dirichlet prior over the document–topic distributions and a symmetric Dirichlet prior over the topic–word distributions results in significantly better model
This PR supports asymmetric alpha in following steps:
To use this new feature, please just run with an extra option "-num_alpha_iterations".
Please notice that there are two TODOs. One is Evaluation in asymmetric prior mode, the other is Inference with asymmetric prior.