Changeset 12137 for trunk/sources

Timestamp:

03/05/15 11:25:59 (10 years ago)

Author:

gkronber

Message:

#2352: only show correlations to 3 digits accuracy.

Location:

trunk/sources

Files:

• HeuristicLab.Analysis.Statistics.Views/3.3/CorrelationView.cs (modified) (1 diff)
• HeuristicLab.Problems.DataAnalysis.Views/3.4/FeatureCorrelation/FeatureCorrelationCalculator.cs (modified) (1 diff)

trunk/sources/HeuristicLab.Analysis.Statistics.Views/3.3/CorrelationView.cs

@@ -214 +214 @@
                 .Where(x => !double.IsNaN(x) && !double.IsNegativeInfinity(x) && !double.IsPositiveInfinity(x));
 
-          if (!rowValues.Any() || !columnValues.Any() || i == j || rowValues.Count() != columnValues.Count()) {
+          if (!rowValues.Any() || !columnValues.Any() || rowValues.Count() != columnValues.Count()) {
             dt[i, j] = double.NaN;
+          } else if (i == j) {
+            dt[i, j] = 1.0;
           } else {
             if (methodName == PearsonName) {
-              dt[i, j] = alglib.pearsoncorr2(rowValues.ToArray(), columnValues.ToArray());
+              dt[i, j] = Math.Round(alglib.pearsoncorr2(rowValues.ToArray(), columnValues.ToArray()), 3); // for correlations it is usually sufficient to show only 3 digits
             } else {
-              dt[i, j] = alglib.spearmancorr2(rowValues.ToArray(), columnValues.ToArray());
+              dt[i, j] = Math.Round(alglib.spearmancorr2(rowValues.ToArray(), columnValues.ToArray()), 3); // for correlations it is usually sufficient to show only 3 digits
             }
           }

trunk/sources/HeuristicLab.Problems.DataAnalysis.Views/3.4/FeatureCorrelation/FeatureCorrelationCalculator.cs

@@ -139 +139 @@
           IEnumerable<double> var2 = problemData.Dataset.GetDoubleValues(doubleVariableNames[j], indices);
 
-          elements[i, j] = calc.Calculate(var1, var2, out error);
+          elements[i, j] = Math.Round(calc.Calculate(var1, var2, out error), 3); // only show correlations to 3 digits accuracy
 
           if (!error.Equals(OnlineCalculatorError.None)) {