minor fixes

MeanShift.h

@@ -11,7 +11,7 @@ struct Cluster {
 class MeanShift {
 public:
     MeanShift() { set_kernel(NULL); }
-    MeanShift(double (*_kernel_func)(double,double)) { set_kernel(kernel_func); }
+    MeanShift(double (*_kernel_func)(double,double)) { set_kernel(_kernel_func); }
     std::vector<std::vector<double> > meanshift(const std::vector<std::vector<double> > &, double);
     std::vector<Cluster> cluster(const std::vector<std::vector<double> > &, double);
 

README.md

@@ -14,9 +14,9 @@ To cluster a set of points, create a MeanShift object and call the `cluster` met
 ## Example
 ```cpp
 vector<vector<double> > points = load_points("test_simple.csv");
-MeanShift *ms = new MeanShift(NULL);
+MeanShift *ms = new MeanShift();
 double kernel_bandwidth = 2;
-vector<vector<double> > shifted_points = ms->cluster(points, 2);
+vector<Cluster > clusters = ms->cluster(points, 2);
 ```
 
 ## Visualization for Linux
@@ -36,7 +36,7 @@ The program will generate a csv file named "result.csv".
 Plot it
 ```bash
 gnuplot
-plot 'test.csv' with points, 'result.csv' with points
+plot 'test.csv' using 1:2 "%lf,%lf", 'result.csv' using 1:2 "%lf,%lf"
 ```
 
 ## License

cpp_test.cpp

@@ -61,11 +61,11 @@ int main(int argc, char **argv)
       for(int point = 0; point < clusters[cluster].original_points.size(); point++){
         for(int dim = 0; dim < clusters[cluster].original_points[point].size(); dim++) {
           printf("%f ", clusters[cluster].original_points[point][dim]);
-          fprintf(fp, dim?",%f":"%f", clusters[cluster].original_points[point][dim]);
         }
         printf(" -> ");
         for(int dim = 0; dim < clusters[cluster].shifted_points[point].size(); dim++) {
           printf("%f ", clusters[cluster].shifted_points[point][dim]);
+          fprintf(fp, dim?",%f":"%f", clusters[cluster].shifted_points[point][dim]);
         }
         printf("\n");
         fprintf(fp, "\n");