Skip to content

Commit

Permalink
clean up language error and fluffy comments
Browse files Browse the repository at this point in the history
  • Loading branch information
MikeLing committed Feb 18, 2017
1 parent 988fe1e commit 937cb42
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 19 deletions.
5 changes: 1 addition & 4 deletions src/shogun/multiclass/BruteKNNSolver.h
Expand Up @@ -15,10 +15,7 @@
namespace shogun
{

/**
* The BruteKNNSolver class is a solver class which inherit from KNNSolver class. It will use brute way
* to classify the object which mean compare the object against all training objects for each prediction.
*/
/* Standard KNN solver. Test points are compared to all training data for each prediction. */
class CBruteKNNSolver : public CKNNSolver
{
public:
Expand Down
5 changes: 2 additions & 3 deletions src/shogun/multiclass/CoverTreeKNNSolver.h
Expand Up @@ -17,9 +17,8 @@ namespace shogun
{

/**
* The CoverTreeKNNSolver class is a solver class which inherit from KNNSolver class.
* It use the cover tree to speed up the nearest neighbor search and you can find more
* detail information on https://en.wikipedia.org/wiki/Cover_tree
* Cover tree solver. It uses cover trees to speed up the nearest neighbour computation.
* For more information, see https://en.wikipedia.org/wiki/Cover_tree
*
*/
class CCoverTreeKNNSolver : public CKNNSolver
Expand Down
7 changes: 3 additions & 4 deletions src/shogun/multiclass/KDTreeKNNSolver.h
Expand Up @@ -16,10 +16,9 @@ namespace shogun
{

/**
* The KDTREEKNNSolver class is a solver class which inherit from KNNSolver class.
* It use a k-d tree (short for k-dimensional tree) to delivery the nearest neighbor searches
* involving the multidimensional search key.
* Check https://en.wikipedia.org/wiki/K-d_tree for more detail about K-D tree.
* KD-tree solver. It uses k-d tree (short for k-dimensional tree) to speed up the
* nearest neighbour computation.
* For more information, see https://en.wikipedia.org/wiki/K-d_tree
*
*/
class CKDTREEKNNSolver : public CKNNSolver
Expand Down
6 changes: 1 addition & 5 deletions src/shogun/multiclass/KNNSolver.h
Expand Up @@ -15,11 +15,7 @@

namespace shogun
{
/**
* The KNNSolver class is the virtual base class of BruteKNNSolcer, CoverTreeKNNSolver, KDTreeKNNsolver, LSHKNNSolver.
* The KNNSolver class will include most of Variables which been used all the other Solver and methods been used to choose
* the index of the most frequent class.
*/
/* Virtual base class for all KNN solvers */

class CDistanceMachine;
class CKNNSolver : public CDistanceMachine
Expand Down
5 changes: 2 additions & 3 deletions src/shogun/multiclass/LSHKNNSolver.h
Expand Up @@ -16,9 +16,8 @@ namespace shogun
{

/**
* The LSHKNNSolver class is a solver class which inherit from KNNSolver class.
* It use LSH(short for Locality-sensitive hashing) to do the nearest neighbor search.
* More detail information can be found in https://en.wikipedia.org/wiki/Locality-sensitive_hashing.
* LSH solver. It uses LSH (short for Locality-sensitive hashing) to do the nearest neighbour computation.
* For more information, see https://en.wikipedia.org/wiki/Locality-sensitive_hashing.
*
*/
#ifdef HAVE_CXX11
Expand Down

0 comments on commit 937cb42

Please sign in to comment.