From 66c7ac4d241152e0da8a333e24cf0bdbf1953254 Mon Sep 17 00:00:00 2001
From: Miroslav Lichvar <mlichvar@redhat.com>
Date: Thu, 28 Apr 2011 17:12:39 +0200
Subject: [PATCH] Revert using unweighted variance in weight calculation

This reverts commit 165e6805ab2e46b6fe6b627fcc5fd72d95802c39.
---
 regress.c     | 12 ++++--------
 regress.h     |  1 -
 sourcestats.c | 14 +++++---------
 3 files changed, 9 insertions(+), 18 deletions(-)

diff --git a/regress.c b/regress.c
index e78fafe..1613d55 100644
--- a/regress.c
+++ b/regress.c
@@ -232,7 +232,6 @@ RGR_FindBestRegression
  double *b0,                    /* estimated y axis intercept */
  double *b1,                    /* estimated slope */
  double *s2,                    /* estimated variance of data points */
- double *us2,                   /* estimated unweighted variance of data points */
  
  double *sb0,                   /* estimated standard deviation of
                                    intercept */
@@ -251,7 +250,7 @@ RGR_FindBestRegression
 {
   double P, Q, U, V, W; /* total */
   double resid[MAX_POINTS * REGRESS_RUNS_RATIO];
-  double ss, uss;
+  double ss;
   double a, b, u, ui, aa;
 
   int start, resid_start, nruns, npoints;
@@ -315,20 +314,17 @@ RGR_FindBestRegression
   *b1 = b;
   *b0 = a;
 
-  ss = uss = 0.0;
+  ss = 0.0;
   for (i=start; i<n; i++) {
     ss += resid[i - resid_start]*resid[i - resid_start] / w[i];
-    uss += resid[i - resid_start]*resid[i - resid_start];
   }
 
   npoints = n - start;
-  ss /= npoints - 2;
-  uss /= npoints - 2;
+  ss /= (double)(npoints - 2);
   *sb1 = sqrt(ss / V);
   aa = u * (*sb1);
   *sb0 = sqrt((ss / W) + (aa * aa));
-  *s2 = ss * npoints / W;
-  *us2 = uss;
+  *s2 = ss * (double) npoints / W;
 
   *new_start = start;
   *dof = npoints - 2;
diff --git a/regress.h b/regress.h
index 1c618ae..4c11ca5 100644
--- a/regress.h
+++ b/regress.h
@@ -86,7 +86,6 @@ RGR_FindBestRegression
  double *b0,                    /* estimated y axis intercept */
  double *b1,                    /* estimated slope */
  double *s2,                    /* estimated variance of data points */
- double *us2,                   /* estimated unweighted variance of data points */
  
  double *sb0,                   /* estimated standard deviation of
                                    intercept */
diff --git a/sourcestats.c b/sourcestats.c
index efd295c..9560eac 100644
--- a/sourcestats.c
+++ b/sourcestats.c
@@ -105,9 +105,6 @@ struct SST_Stats_Record {
   /* This is the estimated residual variance of the data points */
   double variance;
 
-  /* This is the estimated unweighted variance of the data points */
-  double uvariance;
-
   /* This array contains the sample epochs, in terms of the local
      clock. */
   struct timeval sample_times[MAX_SAMPLES * REGRESS_RUNS_RATIO];
@@ -193,7 +190,7 @@ SST_CreateInstance(unsigned long refid, IPAddr *addr)
   inst->estimated_offset_sd = 86400.0; /* Assume it's at least within a day! */
   inst->offset_time.tv_sec = 0;
   inst->offset_time.tv_usec = 0;
-  inst->variance = inst->uvariance = 16.0;
+  inst->variance = 16.0;
   inst->nruns = 0;
   return inst;
 }
@@ -367,7 +364,7 @@ find_min_delay_sample(SST_Stats inst)
    time.  E.g. a value of 4 means that we think the standard deviation
    is four times the fluctuation  of the peer distance */
 
-#define SD_TO_DIST_RATIO 1.4
+#define SD_TO_DIST_RATIO 1.0
 
 /* ================================================== */
 /* This function runs the linear regression operation on the data.  It
@@ -385,7 +382,7 @@ SST_DoNewRegression(SST_Stats inst)
 
   int degrees_of_freedom;
   int best_start, times_back_start;
-  double est_intercept, est_slope, est_var, est_uvar, est_intercept_sd, est_slope_sd;
+  double est_intercept, est_slope, est_var, est_intercept_sd, est_slope_sd;
   int i, j, nruns;
   double min_distance;
   double sd_weight, sd;
@@ -408,7 +405,7 @@ SST_DoNewRegression(SST_Stats inst)
 
     /* And now, work out the weight vector */
 
-    sd = sqrt(inst->uvariance);
+    sd = sqrt(inst->variance);
     if (sd > min_distance || sd <= 0.0)
       sd = min_distance;
 
@@ -421,7 +418,7 @@ SST_DoNewRegression(SST_Stats inst)
   inst->regression_ok = RGR_FindBestRegression(times_back + inst->runs_samples,
                                          offsets + inst->runs_samples, weights,
                                          inst->n_samples, inst->runs_samples,
-                                         &est_intercept, &est_slope, &est_var, &est_uvar,
+                                         &est_intercept, &est_slope, &est_var,
                                          &est_intercept_sd, &est_slope_sd,
                                          &best_start, &nruns, &degrees_of_freedom);
 
@@ -436,7 +433,6 @@ SST_DoNewRegression(SST_Stats inst)
     inst->offset_time = inst->sample_times[inst->last_sample];
     inst->estimated_offset_sd = est_intercept_sd;
     inst->variance = est_var;
-    inst->uvariance = est_uvar;
     inst->nruns = nruns;
 
     stress = fabs(old_freq - inst->estimated_frequency) / old_skew;