Changeset 13242

Timestamp:

11/18/15 11:56:07 (9 years ago)

Author:

mkommend

Message:

#1967: Introduced separate lock object for synchronizing access to the random object.

File:

• trunk/sources/HeuristicLab.Algorithms.DataAnalysis/3.4/GaussianProcess/GaussianProcessCovarianceOptimizationProblem.cs (modified) (2 diffs)

trunk/sources/HeuristicLab.Algorithms.DataAnalysis/3.4/GaussianProcess/GaussianProcessCovarianceOptimizationProblem.cs

@@ -192 +192 @@
     }
 
+    private readonly object syncRoot = new object();
     // Does not produce the same result for the same seed when using parallel engine (see below)!
     public override double Evaluate(ISymbolicExpressionTree tree, IRandom random) {
@@ -223 +224 @@
         // Evaluate might be called concurrently therefore access to random has to be synchronized.
         // However, results of multiple runs with the same seed will be different when using the parallel engine.
-        lock (random) {
+        lock (syncRoot) {
           for (int i = 0; i < covarianceFunction.GetNumberOfParameters(nVars); i++) {
             hyperParameters[1 + i] = random.NextDouble() * 2.0 - 1.0;