Changeset 12725 for stable/HeuristicLab.Analysis.Statistics.Views/3.3/StatisticalTestsView.cs

Timestamp:

07/11/15 13:37:32 (9 years ago)

Author:

ascheibe

Message:

#2270 and #2354: merged r12173, r12458, r12077, r12599, r12613, r12112, r12116, r12117, r12131, r12631, r12672, r12684, r12690, r12692 into stable

Location:

stable

Files:

• . (modified) (1 prop)
• HeuristicLab.Analysis.Statistics.Views/3.3/StatisticalTestsView.cs (modified) (20 diffs)

stable/HeuristicLab.Analysis.Statistics.Views/3.3/StatisticalTestsView.cs

@@ -27 +27 @@
 using HeuristicLab.Collections;
 using HeuristicLab.Common;
+using HeuristicLab.Common.Resources;
 using HeuristicLab.Core.Views;
 using HeuristicLab.Data;
@@ -40 +41 @@
     private const int requiredSampleSize = 5;
     private double[][] data;
+    private bool suppressUpdates;
+    private bool initializing;
 
     public double SignificanceLevel {
@@ -76 +79 @@
 
       if (Content != null) {
-        UpdateResultComboBox();
-        UpdateGroupsComboBox();
-        RebuildDataTable();
+        UpdateUI();
+      } else {
+        ResetUI();
       }
       UpdateCaption();
+    }
+
+    private void UpdateUI() {
+      initializing = true;
+      UpdateResultComboBox();
+      UpdateGroupsComboBox();
+      RebuildDataTable();
+      FillCompComboBox();
+      ResetUI();
+      CalculateValues();
+      initializing = false;
     }
 
@@ -92 +106 @@
       Content.ColumnsChanged += Content_ColumnsChanged;
       Content.RowsChanged += Content_RowsChanged;
-      Content.CollectionReset += new CollectionItemsChangedEventHandler<IRun>(Content_CollectionReset);
+      Content.CollectionReset += Content_CollectionReset;
       Content.UpdateOfRunsInProgressChanged += Content_UpdateOfRunsInProgressChanged;
     }
@@ -100 +114 @@
       Content.ColumnsChanged -= Content_ColumnsChanged;
       Content.RowsChanged -= Content_RowsChanged;
-      Content.CollectionReset -= new CollectionItemsChangedEventHandler<IRun>(Content_CollectionReset);
+      Content.CollectionReset -= Content_CollectionReset;
       Content.UpdateOfRunsInProgressChanged -= Content_UpdateOfRunsInProgressChanged;
     }
 
     void Content_RowsChanged(object sender, EventArgs e) {
+      if (suppressUpdates) return;
+      if (InvokeRequired) Invoke((Action<object, EventArgs>)Content_RowsChanged, sender, e);
+      else {
+        UpdateUI();
+      }
+    }
+
+    void Content_ColumnsChanged(object sender, EventArgs e) {
+      if (suppressUpdates) return;
+      if (InvokeRequired) Invoke((Action<object, EventArgs>)Content_ColumnsChanged, sender, e);
+      else {
+        UpdateUI();
+      }
+    }
+
+    private void Content_CollectionReset(object sender, CollectionItemsChangedEventArgs<IRun> e) {
+      if (suppressUpdates) return;
+      if (InvokeRequired) Invoke((Action<object, CollectionItemsChangedEventArgs<IRun>>)Content_CollectionReset, sender, e);
+      else {
+        UpdateUI();
+      }
+    }
+
+    void Content_UpdateOfRunsInProgressChanged(object sender, EventArgs e) {
+      if (InvokeRequired) Invoke((Action<object, EventArgs>)Content_UpdateOfRunsInProgressChanged, sender, e);
+      else {
+        suppressUpdates = Content.UpdateOfRunsInProgress;
+        if (!suppressUpdates) UpdateUI();
+      }
+    }
+
+    private void openBoxPlotToolStripMenuItem_Click(object sender, EventArgs e) {
+      RunCollectionBoxPlotView boxplotView = new RunCollectionBoxPlotView();
+      boxplotView.Content = Content;
+      boxplotView.SetXAxis(groupComboBox.SelectedItem.ToString());
+      boxplotView.SetYAxis(resultComboBox.SelectedItem.ToString());
+
+      boxplotView.Show();
+    }
+
+    private void groupCompComboBox_SelectedValueChanged(object sender, EventArgs e) {
+      if (initializing || suppressUpdates) return;
+      string curItem = (string)groupCompComboBox.SelectedItem;
+      CalculatePairwise(curItem);
+    }
+
+    private void resultComboBox_SelectedValueChanged(object sender, EventArgs e) {
+      if (initializing || suppressUpdates) return;
       RebuildDataTable();
-    }
-
-    void Content_ColumnsChanged(object sender, EventArgs e) {
+      ResetUI();
+      CalculateValues();
+    }
+
+    private void groupComboBox_SelectedValueChanged(object sender, EventArgs e) {
+      if (initializing || suppressUpdates) return;
       RebuildDataTable();
-    }
-
-    private void Content_CollectionReset(object sender, CollectionItemsChangedEventArgs<IRun> e) {
-      RebuildDataTable();
-    }
-
-    void Content_UpdateOfRunsInProgressChanged(object sender, EventArgs e) {
-      if (!Content.UpdateOfRunsInProgress) {
-        RebuildDataTable();
-      }
+      FillCompComboBox();
+      ResetUI();
+      CalculateValues();
     }
     #endregion
 
     private void UpdateGroupsComboBox() {
+      string selectedItem = (string)groupComboBox.SelectedItem;
+
       groupComboBox.Items.Clear();
-
       var parameters = (from run in Content
                         where run.Visible
@@ -154 +213 @@
         }
 
-        if (possibleIndizes.Count > 0) {
+        if (selectedItem != null && groupComboBox.Items.Contains(selectedItem)) {
+          groupComboBox.SelectedItem = selectedItem;
+        } else if (possibleIndizes.Count > 0) {
           groupComboBox.SelectedItem = groupComboBox.Items[possibleIndizes.First()];
-        } else {
-          groupComboBox.SelectedItem = groupComboBox.Items[0];
         }
       }
@@ -169 +228 @@
 
     private void UpdateResultComboBox() {
+      string selectedItem = (string)resultComboBox.SelectedItem;
+
       resultComboBox.Items.Clear();
       var results = (from run in Content
@@ -177 +238 @@
 
       resultComboBox.Items.AddRange(results);
-      if (resultComboBox.Items.Count > 0) resultComboBox.SelectedItem = resultComboBox.Items[0];
+
+      if (selectedItem != null && resultComboBox.Items.Contains(selectedItem)) {
+        resultComboBox.SelectedItem = selectedItem;
+      } else if (resultComboBox.Items.Count > 0) {
+        resultComboBox.SelectedItem = resultComboBox.Items[0];
+      }
     }
 
     private void FillCompComboBox() {
+      string selectedItem = (string)groupCompComboBox.SelectedItem;
       string parameterName = (string)groupComboBox.SelectedItem;
       if (parameterName != null) {
@@ -189 +256 @@
           groupCompComboBox.Items.Clear();
           columnNames.ForEach(x => groupCompComboBox.Items.Add(x));
-          if (groupCompComboBox.Items.Count > 0) groupCompComboBox.SelectedItem = groupCompComboBox.Items[0];
+          if (selectedItem != null && groupCompComboBox.Items.Contains(selectedItem)) {
+            groupCompComboBox.SelectedItem = selectedItem;
+          } else if (groupCompComboBox.Items.Count > 0) {
+            groupCompComboBox.SelectedItem = groupCompComboBox.Items[0];
+          }
         }
       }
@@ -259 +330 @@
     }
 
-    private void resultComboBox_SelectedValueChanged(object sender, EventArgs e) {
-      RebuildDataTable();
-      ResetUI();
-      CalculateValues();
-    }
-
-    private void groupComboBox_SelectedValueChanged(object sender, EventArgs e) {
-      RebuildDataTable();
-      FillCompComboBox();
-      ResetUI();
-      CalculateValues();
-    }
-
     private bool VerifyDataLength(bool showMessage) {
       if (data == null || data.Length == 0)
@@ -277 +335 @@
 
       //alglib needs at least 5 samples for computation
-      if (data.Any(x => x.Length <= requiredSampleSize)) {
+      if (data.Any(x => x.Length < requiredSampleSize)) {
         if (showMessage)
           MessageBox.Show(this, "You need at least " + requiredSampleSize
@@ -291 +349 @@
         return;
 
-      if (data != null) {
-        MainFormManager.GetMainForm<HeuristicLab.MainForm.WindowsForms.MainForm>()
+      if (data != null && data.All(x => x != null)) {
+        MainFormManager.GetMainForm<MainForm.WindowsForms.MainForm>()
           .AddOperationProgressToView(this, "Calculating...");
 
@@ -305 +363 @@
       CalculatePairwiseTest(groupName);
 
-      MainFormManager.GetMainForm<HeuristicLab.MainForm.WindowsForms.MainForm>().RemoveOperationProgressFromView(this);
+      MainFormManager.GetMainForm<MainForm.WindowsForms.MainForm>().RemoveOperationProgressFromView(this);
     }
 
     private void CalculatePairwise(string groupName) {
+      if (groupName == null) return;
       if (!VerifyDataLength(false))
         return;
 
-      MainFormManager.GetMainForm<HeuristicLab.MainForm.WindowsForms.MainForm>().AddOperationProgressToView(pairwiseTestGroupBox, "Calculating...");
+      MainFormManager.GetMainForm<MainForm.WindowsForms.MainForm>().AddOperationProgressToView(pairwiseTestGroupBox, "Calculating...");
       Task.Factory.StartNew(() => CalculatePairwiseAsync(groupName));
     }
@@ -319 +378 @@
       CalculatePairwiseTest(groupName);
 
-      MainFormManager.GetMainForm<HeuristicLab.MainForm.WindowsForms.MainForm>().RemoveOperationProgressFromView(pairwiseTestGroupBox);
+      MainFormManager.GetMainForm<MainForm.WindowsForms.MainForm>().RemoveOperationProgressFromView(pairwiseTestGroupBox);
     }
 
     private void CalculateAllGroupsTest() {
       double pval = KruskalWallisTest.Test(data);
-      pValTextBox.Text = pval.ToString();
-      if (pval < significanceLevel) {
-        this.Invoke(new Action(() => {
-          groupCompLabel.Image = HeuristicLab.Common.Resources.VSImageLibrary.Default;
+      DisplayAllGroupsTextResults(pval);
+    }
+
+    private void DisplayAllGroupsTextResults(double pval) {
+      if (InvokeRequired) {
+        Invoke((Action<double>)DisplayAllGroupsTextResults, pval);
+      } else {
+        pValTextBox.Text = pval.ToString();
+        if (pval < significanceLevel) {
+          groupCompLabel.Image = VSImageLibrary.Default;
           groupComTextLabel.Text = "There are groups with different distributions";
-        }));
-      } else {
-        this.Invoke(new Action(() => {
-          groupCompLabel.Image = HeuristicLab.Common.Resources.VSImageLibrary.Warning;
+        } else {
+          groupCompLabel.Image = VSImageLibrary.Warning;
           groupComTextLabel.Text = "Groups have an equal distribution";
-        }));
+        }
       }
     }
@@ -343 +406 @@
       DoubleMatrix pValsMatrix = new DoubleMatrix(1, stringConvertibleMatrixView.Content.Columns);
       pValsMatrix.ColumnNames = stringConvertibleMatrixView.Content.ColumnNames;
-      pValsMatrix.RowNames = new string[] { "p-Value" };
+      pValsMatrix.RowNames = new[] { "p-Value" };
 
       for (int i = 0; i < data.Length; i++) {
@@ -353 +416 @@
       // p-value is below significance level and thus the null hypothesis (data is normally distributed) is rejected
       if (res.Any(x => x < significanceLevel)) {
-        this.Invoke(new Action(() => {
-          normalityLabel.Image = HeuristicLab.Common.Resources.VSImageLibrary.Warning;
+        Invoke(new Action(() => {
+          normalityLabel.Image = VSImageLibrary.Warning;
           normalityTextLabel.Text = "Some groups may not be normally distributed";
         }));
       } else {
-        this.Invoke(new Action(() => {
-          normalityLabel.Image = HeuristicLab.Common.Resources.VSImageLibrary.Default;
+        Invoke(new Action(() => {
+          normalityLabel.Image = VSImageLibrary.Default;
           normalityTextLabel.Text = "All sample data is normally distributed";
         }));
       }
 
-      this.Invoke(new Action(() => {
+      Invoke(new Action(() => {
         normalityStringConvertibleMatrixView.Content = pValsMatrix;
         normalityStringConvertibleMatrixView.DataGridView.AutoResizeColumns(DataGridViewAutoSizeColumnsMode.AllCells);
@@ -372 +435 @@
     private void ShowPairwiseResult(int nrOfEqualDistributions) {
       double ratio = ((double)nrOfEqualDistributions) / (data.Length - 1) * 100.0;
-      equalDistsTextBox.Text = ratio.ToString() + " %";
+      equalDistsTextBox.Text = ratio + " %";
 
       if (nrOfEqualDistributions == 0) {
-        this.Invoke(new Action(() => {
-          pairwiseLabel.Image = HeuristicLab.Common.Resources.VSImageLibrary.Default;
+        Invoke(new Action(() => {
+          pairwiseLabel.Image = VSImageLibrary.Default;
           pairwiseTextLabel.Text = "All groups have different distributions";
         }));
       } else {
-        this.Invoke(new Action(() => {
-          pairwiseLabel.Image = HeuristicLab.Common.Resources.VSImageLibrary.Warning;
+        Invoke(new Action(() => {
+          pairwiseLabel.Image = VSImageLibrary.Warning;
           pairwiseTextLabel.Text = "Some groups have equal distributions";
         }));
@@ -394 +457 @@
       double[][] newData = FilterDataForPairwiseTest(colIndex);
 
-      var rowNames = new string[] { "p-Value of Mann-Whitney U", "Adjusted p-Value of Mann-Whitney U", 
+      var rowNames = new[] { "p-Value of Mann-Whitney U", "Adjusted p-Value of Mann-Whitney U", 
             "p-Value of T-Test", "Adjusted p-Value of T-Test", "Cohen's d", "Hedges' g" };
 
@@ -433 +496 @@
       }
 
-      this.Invoke(new Action(() => {
+      Invoke(new Action(() => {
         pairwiseStringConvertibleMatrixView.Content = pValsMatrix;
         pairwiseStringConvertibleMatrixView.DataGridView.AutoResizeColumns(DataGridViewAutoSizeColumnsMode.AllCells);
@@ -469 +532 @@
       return newData;
     }
-
-    private void openBoxPlotToolStripMenuItem_Click(object sender, EventArgs e) {
-      RunCollectionBoxPlotView boxplotView = new RunCollectionBoxPlotView();
-      boxplotView.Content = Content;
-      boxplotView.SetXAxis(groupComboBox.SelectedItem.ToString());
-      boxplotView.SetYAxis(resultComboBox.SelectedItem.ToString());
-
-      boxplotView.Show();
-    }
-
-    private void groupCompComboBox_SelectedValueChanged(object sender, EventArgs e) {
-      string curItem = (string)groupCompComboBox.SelectedItem;
-      CalculatePairwise(curItem);
-    }
   }
 }