search

nateraw / Lda2vec-Tensorflow

Tensorflow 1.5 implementation of Chris Moody's Lda2vec, adapted from @meereeum
MIT License
107 stars 40 forks source link

Lda2vec model should be split into two versions - Single and Multi-Context #22

Closed nateraw closed 5 years ago

nateraw commented 5 years ago

Currently, the model handles multi-context lda2vec. That's cool, but it makes the regular version extremely hard to read/understand. Making 2 versions and/or subclassing the original model would allow for the regular version to be more understandable to new users.

nateraw commented 5 years ago

Model is now only single context. Multi-context will remain out of scope for now, as it doesn't really work anyways (as noted by the author himself).