search

buliugu / airhead-research

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

Code review request #49

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Purpose of code changes on this branch:
 - Update the matrix transformations
 - A new utility class for gathering the most frequently used statistics
used for transformations
 - A new interface for transformations that only use global statistics
    - Use this interface for doing all current transformations. This new
way should make it significantly easier to write new transformations by
separating the iteration method for stat gathering, and the transformation
method
    - This also fixes several bugs in some of the transformations, e.g.,
some paths in the TF-IDF transform
  - Simple unit tests are provided for each transformation are in the test
path  

When reviewing my code changes, please focus on
 - new interface design
 - Comments
 - New transformations, structure and correctness

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

Original issue reported on code.google.com by FozzietheBeat@gmail.com on 10 Apr 2010 at 8:52

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

Original comment by FozzietheBeat@gmail.com on 21 Jul 2010 at 1:48