search

buliugu / airhead-research

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

Term Document space refactoring #82

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Purpose of code changes on this branch:
  The three main term document vector spaces have a lot of code duplication, which can lead to variations in run time efficiency and correctness.  This change factors out as much similarity into one single class.  The original vector spaces now let the base class do most of the work while retaining the same interface.

This should also address Andre's concerns regarding ESA.

When reviewing my code changes, please focus on:
Code factorization.  Potential extensions that are needed.

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

Original issue reported on code.google.com by FozzietheBeat@gmail.com on 27 Jan 2011 at 9:49

GoogleCodeExporter commented 9 years ago 
Committed! with all unit tests passing!

Original comment by FozzietheBeat@gmail.com on 26 Feb 2011 at 3:49