Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-12-02 Thread Jake Mannix
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/ --- (Updated 2011-12-02 20:49:52.055735) Review request for mahout and Ted Dunning.

Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-30 Thread Jake Mannix
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/ --- (Updated 2011-12-01 03:44:25.140987) Review request for mahout and Ted Dunning.

Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-30 Thread Jake Mannix
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/ --- (Updated 2011-11-30 18:42:21.747592) Review request for mahout and Ted Dunning.

Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-30 Thread Jake Mannix
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/ --- (Updated 2011-11-30 08:58:25.710229) Review request for mahout and Ted Dunning.

Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-29 Thread Jake Mannix
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/ --- (Updated 2011-11-30 07:35:21.576144) Review request for mahout and Ted Dunning.

Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-28 Thread Jake Mannix
> On 2011-11-28 07:54:29, Ted Dunning wrote: > > Generally this looks like pretty clean code. Some more comments about > > intent would be nice. > > > > My review so far is very superficial. I'm pretty blind to places which need more docs, as it all does, and I know the code. If you could p

Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-28 Thread Jake Mannix
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/#review3538 --- trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/Caching

Re: Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-28 Thread Ted Dunning
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/#review3531 --- Generally this looks like pretty clean code. Some more comments about

Review Request: New implementation for LDA: Collapsed Variational Bayes (0th derivative approximation), with map-side model caching

2011-11-27 Thread Jake Mannix
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2944/ --- Review request for mahout. Summary --- See MAHOUT-897 This addresses bug