Ldaresults - Githubissues

columbia-applied-data-science / rosetta

Tools, wrappers, etc... for data science with a concentration on text processing

Other

206 stars 47 forks source link

Closed dkrasner closed 9 years ago

dkrasner commented 9 years ago

Some cleanup:

removed redundant probability data frame in LDAResults * cleaned up and made more uniform
removed catdoc from list of unix file converter utils since it's no longer supported and fails on OSX; replaced it with antiword utility
test cleanup to reflect above

langmore commented 9 years ago

I had a few comments. Use what you like. LGTM.

dkrasner commented 9 years ago

@langmore ok let me know if this looks ok to merge

langmore commented 9 years ago

LGTM (= "Looks good to me", so go ahead and merge after making/not making the suggested changes).

dkrasner commented 9 years ago

merging

@langmore - maybe we are due for a micro/small release?