Changeset 13312 for stable/HeuristicLab.Problems.GeneticProgramming/3.3/robocode/Interpreter.cs

Timestamp:

11/19/15 21:16:55 (8 years ago)

Author:

gkronber

Message:

#2069: merged r13210,r13225,r13266,r13309,r13311 from trunk to stable

Location:

stable

Files:

• . (modified) (1 prop)
• HeuristicLab.Problems.GeneticProgramming/3.3/robocode (copied) (copied from trunk/sources/HeuristicLab.Problems.GeneticProgramming/3.3/robocode)
• HeuristicLab.Problems.GeneticProgramming/3.3/robocode/Interpreter.cs (modified) (6 diffs)

stable/HeuristicLab.Problems.GeneticProgramming/3.3/robocode/Interpreter.cs

@@ -31 +31 @@
 namespace HeuristicLab.Problems.GeneticProgramming.Robocode {
   public static class Interpreter {
+
+    // necessary for synchronization to guarantee that only one robocode program is executed at the same time
+    // NOTE: this does not guarantee OS-wide mutual exclusion, but we ignore that for now
+    private static readonly object syncRoot = new object();
+
     // TODO performance: it would probably be useful to implement the BattleRunner in such a way that we don't have to restart the java process each time, e.g. using console IO to load & run robots 
     public static double EvaluateTankProgram(ISymbolicExpressionTree tree, string path, EnemyCollection enemies, string robotName = null, bool showUI = false, int nrOfRounds = 3) {
@@ -56 +61 @@
       javaCompileInfo.CreateNoWindow = true;
 
+      // it's ok to compile multiple robocode programs concurrently
       using (Process javaCompile = new Process()) {
         javaCompile.StartInfo = javaCompileInfo;
@@ -70 +76 @@
       }
 
-      //parallel execution of multiple robocode instances can sometimes lead to a damaged robot.database
-      var robotsDbFileName = Path.Combine(path, "robots", "robot.database");
-      if (File.Exists(robotsDbFileName))
-        File.Delete(robotsDbFileName);
-
       ProcessStartInfo evaluateCodeInfo = new ProcessStartInfo();
 
@@ -89 +90 @@
       evaluateCodeInfo.CreateNoWindow = true;
 
+      // the robocode framework writes state to a file therefore parallel evaluation of multiple robocode programs is not possible yet.
       double evaluation;
-      using (Process evaluateCode = new Process()) {
-        evaluateCode.StartInfo = evaluateCodeInfo;
-        evaluateCode.Start();
-        evaluateCode.WaitForExit();
-
-        if (evaluateCode.ExitCode != 0) {
-          DeleteRobotFiles(path, robotName);
-          throw new Exception("Error running Robocode: " + evaluateCode.StandardError.ReadToEnd() + Environment.NewLine +
-                              evaluateCode.StandardOutput.ReadToEnd());
-        }
-
-        try {
-          string scoreString =
-            evaluateCode.StandardOutput.ReadToEnd()
-              .Split(new string[] { Environment.NewLine }, StringSplitOptions.RemoveEmptyEntries)
-              .Last();
-          evaluation = Double.Parse(scoreString, CultureInfo.InvariantCulture);
-        }
-        catch (Exception ex) {
-          throw new Exception("Error parsing score string: " + ex);
-        }
-        finally {
-          DeleteRobotFiles(path, robotName);
+      lock (syncRoot) {
+        using (Process evaluateCode = new Process()) {
+          evaluateCode.StartInfo = evaluateCodeInfo;
+          evaluateCode.Start();
+          evaluateCode.WaitForExit();
+
+          if (evaluateCode.ExitCode != 0) {
+            DeleteRobotFiles(path, robotName);
+            throw new Exception("Error running Robocode: " + evaluateCode.StandardError.ReadToEnd() +
+                                Environment.NewLine +
+                                evaluateCode.StandardOutput.ReadToEnd());
+          }
+
+          try {
+            string scoreString =
+              evaluateCode.StandardOutput.ReadToEnd()
+                .Split(new string[] { Environment.NewLine }, StringSplitOptions.RemoveEmptyEntries)
+                .Last();
+            evaluation = Double.Parse(scoreString, CultureInfo.InvariantCulture);
+          }
+          catch (Exception ex) {
+            throw new Exception("Error parsing score string: " + ex);
+          }
+          finally {
+            DeleteRobotFiles(path, robotName);
+          }
         }
       }
@@ -222 +227 @@
 
     private static string InterpretCodeSymbol(ISymbolicExpressionTreeNode node) {
-      var sy = node.Symbol as CodeSymbol;
-      string code = string.Join(Environment.NewLine, node.Subtrees.Select(Interpret));
-      return string.Format(
-@"{0}
-{1}
-{2}", sy.Prefix, code, sy.Suffix);
+      var sy = (CodeSymbol)node.Symbol;
+      string code = string.Join(Environment.NewLine, node.Subtrees.Select(Interpret));
+      return sy.Prefix + Environment.NewLine + code + Environment.NewLine + sy.Suffix;
     }
 
     private static string InterpretBoolValue(ISymbolicExpressionTreeNode node) {
-      var boolNode = node as BooleanTreeNode;
+      var boolNode = (BooleanTreeNode)node;
       return string.Format(NumberFormatInfo.InvariantInfo, "{0}", boolNode.Value).ToLower();
     }
 
     private static string InterpretNumber(ISymbolicExpressionTreeNode node) {
-      var numberNode = node as NumberTreeNode;
+      var numberNode = (NumberTreeNode)node;
       return string.Format(NumberFormatInfo.InvariantInfo, "{0}", numberNode.Value);
     }
 
     private static string InterpetShotPower(ISymbolicExpressionTreeNode node) {
-      var shotPowerNode = node as ShotPowerTreeNode;
+      var shotPowerNode = (ShotPowerTreeNode)node;
       return string.Format(NumberFormatInfo.InvariantInfo, "{0:E}", shotPowerNode.Value);
     }
@@ -306 +308 @@
       rhs = node.GetSubtree(1);
 
-      return Interpret(lhs) + " == " + Interpret(rhs);
+      return Interpret(lhs) + compSy + Interpret(rhs);
     }