JohnLetnev / airhead-research

Automatically exported from code.google.com/p/airhead-research
0 stars 0 forks source link

Wordsi, once and for all #90

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Purpose of code changes on this branch:
Check in the Wordsi code, a refactoring of the basis mapping, and a slew of 
clustering implementations.  This also includes the refactoring of the matrix 
transforms.

It's a really massive change, so i'm also open to breaking this up into smaller 
chunks.

When reviewing my code changes, please focus on:
Check the interface for wordsi.  Is each component's purpose and implementation 
clear?  Is there a better abstraction that keeps the code minimal?
Check the clustering implementations.  Does the online streaming interface look 
good or is it missing pieces?
For the basis mapping, part of this has been pushed into the trunk already, so 
just think if there are any other ways to clean up the interface and code

The matrix transforms haven't been changed significantly, but they should get 
put into the trunk at some point since this implementation is a bit easier to 
deal with than the current one.

After the review, I'll merge this branch into:
/trunk

Original issue reported on code.google.com by FozzietheBeat@gmail.com on 30 Mar 2011 at 11:20

GoogleCodeExporter commented 9 years ago
Set review issue status to: Done

Original comment by FozzietheBeat@gmail.com on 16 Sep 2011 at 5:51