Create experiment dialog throws an exception if more than one instance provider does not have any data descriptors

[mkommend]

The problem is an error in the progress calculation during the provider discovery. If multiple instance provider do not have any data descriptors a progress of > 100 is reported, that leads to an exception in the progress bar control (CreateExperimentDialog line 734).

Progress calculation (line 725-726)

 double progress = nodes.Count > 0 ? i / (double)nodes.Count : 0.0;
 instanceDiscoveryBackgroundWorker.ReportProgress((int)(100 * progress), provider.Name);

The issues is that i can be larger then nodes.Count due to excluded instance providers.

[mkommend]

r13671: Changed progress calculationd during instance provider discovery in CreateExperimentDialog.

[abeham]

Change r13671 is okay, there's a lot of formatting changes which obscures the real cange. As a consistent extension of how we treat if-else conditions, I would urge to treat catch/finally in a similar way:

if (Maximization.Value) {
  // largest values must be sorted first
  trainingQuality = Quality.Select(x => -x.Value).ToArray();
} else {
  // smallest values must be sorted first
  trainingQuality = Quality.Select(x => x.Value).ToArray();
}

try (...) {
  ...
} finally {
  ...
}

[mkommend]

r13885: Merged r13671 into stable.