Fix issue with empty hall of fame

benchmark.jl

@@ -6,14 +6,17 @@ fullRun(3,
     annealing=true,
     ncyclesperiteration=100,
     fractionReplaced=0.1f0,
-    verbosity=0)
+    verbosity=0,
+    topn=10
+)
 
 t = @benchmark(fullRun(3,
         npop=100,
         annealing=true,
         ncyclesperiteration=100,
         fractionReplaced=0.1f0,
-        verbosity=0
+        verbosity=0,
+        topn=10
        ), evals=10)
 
 tnoanneal = @benchmark(fullRun(3,
@@ -21,7 +24,8 @@ tnoanneal = @benchmark(fullRun(3,
         annealing=false,
         ncyclesperiteration=100,
         fractionReplaced=0.1f0,
-        verbosity=0
+        verbosity=0,
+        topn=10
        ), evals=10)
 
 println("The median time is $(median(t)) with annealing, $(median(tnoanneal)) without")

eureqa.jl

@@ -415,7 +415,7 @@ function bestOfSample(pop::Population)::PopMember
 end
 
 # Return best 10 examples
-function bestSubPop(pop::Population; topn::Int32=10)::Population
+function bestSubPop(pop::Population; topn::Integer=10)::Population
     best_idx = sortperm([pop.members[member].score for member=1:pop.n])
     return Population(pop.members[best_idx[1:topn]])
 end

paralleleureqa.jl

@@ -19,7 +19,7 @@ function fullRun(niterations::Integer;
                 ncyclesperiteration::Integer=3000,
                 fractionReplaced::Float32=0.1f0,
                 verbosity::Integer=0,
-                topn::Int32=10
+                topn::Integer=10
                )
     debug(verbosity, "Lets try to learn (x2^2 + cos(x3)) using regularized evolution from scratch")
     debug(verbosity, "Running with $nthreads threads")
@@ -82,11 +82,11 @@ function fullRun(niterations::Integer;
         end
 
         # Hall of fame migration
-        if hofMigration
+        if hofMigration && size(dominating)[1] > 0
             for j=1:nthreads
                 for k in rand(1:npop, Integer(npop*fractionReplacedHof))
                     # Copy in case one gets used twice
-                    allPops[j].members[k] = deepcopy(dominating[rand(2:size(dominating)[1])])
+                    allPops[j].members[k] = deepcopy(dominating[rand(1:size(dominating)[1])])
                 end
             end
         end