New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching
Review Request #2944 - Created Nov. 27, 2011 and updated
mvn clean test
Generally this looks like pretty clean code. Some more comments about intent would be nice. My review so far is very superficial.
Why return double? Main ignores this.
I think convention is @override on a seaparate line.
This comment is very confusing.
Javadoc here would be nice. Why is this sampler different from samplers we already have? Also, I don't see test code for this.
So this looks like a multinomial sampler. Why not fit it into what already exists?
Adds appropriate ASL headers to new files, adds a bunch of nice javadocs, some TODOs to clean up some detritus, moves MemoryUtil to a more common location. Could use some more review if anyone has an urge, but otherwise this code is ready to go. More updates can come in the future. And more docs, etc.