cemoody / lda2vec

MIT License
3.15k stars 628 forks source link

Documentation outdated #44

Open Eleonore9 opened 8 years ago

Eleonore9 commented 8 years ago

The Readme for the hacker_news example refers to a model.py and visualize.py. I couldn't find those files.

radekrepo commented 8 years ago

Those files don't actually exist under the name suggested by the Readme. I found this inconsistency frustrating when trying my own code.

For the hackernews example, some models can be found in the directory: lda2vec/examples/hacker_news/lda2vec/

There is no visualization provided for the hackernews example. The only visualization example with lda2vec results that I'm aware of is here: http://nbviewer.jupyter.org/github/cemoody/lda2vec/blob/master/examples/twenty_newsgroups/lda2vec/lda2vec.ipynb

Eleonore9 commented 8 years ago

Thanks a lot @nanader! I'd like to try and update the docs, then! I think it'd be useful for other people.