-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds scores for topic distributions #3
base: master
Are you sure you want to change the base?
Conversation
slda.cpp
Outdated
@@ -986,6 +989,7 @@ void slda::infer_only(corpus * c, const settings * setting, const char * directo | |||
{ | |||
score += eta[i][k] * phi_m[k]; | |||
} | |||
fprintf(inf_score_file, "%5.5f ", score); |
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.
suggest also adding doc id for the easy output understanding.
indent is not correct.
slda.cpp
Outdated
@@ -994,7 +998,7 @@ void slda::infer_only(corpus * c, const settings * setting, const char * directo | |||
} | |||
if (label == doc->label) | |||
num_correct ++; | |||
|
|||
fprintf(inf_score_file, "\n"); |
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.
indent is not correct.
Added doc_id and corrected the indent |
@chongw can you review my latest commit? |
The scores are essential for determining the near best fits of various topics in the SLDA. Please advise if another technique needs to be employed or if this looks good.