Simplify Population#new_evolvables method by removing the evolvables parameter

Author: mattruzicka

CHANGELOG.md

@@ -8,6 +8,7 @@ Breaking Changes:
 * Renamed `Evolvable#after_initialize` to `Evolvable#after_initialize_evolvable`.
 * Changed population initialization and evolvable creation workflow for more consistency.
 * Simplified error handling in favor of more single  `Evolvable::Error` type with more informative messages.
+* Changed `Population#new_evolvables` to only accept a `count` parameter, removing the `evolvables` parameter.
 
 New Features:
 * Added gene clusters for organizing related genes through the `cluster` parameter in gene definitions.

lib/evolvable/population.rb

@@ -107,7 +107,8 @@ def initialize(evolvable_type:,
       self.selection = selection if selection
       self.combination = combination if combination
       self.mutation = mutation if mutation
-      @evolvables = new_evolvables(count: @size - evolvables.count, evolvables: evolvables)
+      self.evolvables = evolvables || []
+      new_evolvables(count: @size - evolvables.count)
     end
 
     #
@@ -256,17 +257,16 @@ def new_evolvable(genome: nil)
     # Creates multiple new evolvable instances.
     #
     # @param count [Integer] The number of evolvables to create
-    # @param evolvables [Array<Evolvable>, nil] Optional existing evolvables to include
-    # @return [Array<Evolvable>] The collection of evolvables
+    # @return [Array<Evolvable>] The newly created evolvables
     #
-    def new_evolvables(count:, evolvables: nil)
-      evolvables ||= @evolvables || []
-      @evolvables = evolvables
-
+    def new_evolvables(count:)
       if parent_evolvables.empty?
         Array.new(count) { new_evolvable(genome: new_genome) }
       else
-        @evolvables = generate_evolvables(count)
+        evolvables = generate_evolvables(count)
+        @evolvables ||= []
+        @evolvables.concat evolvables
+        evolvables
       end
     end