Changeset 1148 for framspy

Timestamp:

08/10/21 00:48:18 (3 years ago)

Author:

Maciej Komosinski

Message:

Added exception when popsize is not a multiple of 4 (required by deap.tools.selTournamentDCD())

Location:

framspy/evolalg

Files:

• examples/multicriteria.py (modified) (3 diffs)
• selection/nsga2.py (modified) (1 diff)

framspy/evolalg/examples/multicriteria.py

@@ -168 +168 @@
 def create_experiment():
     parsed_args = parseArguments()
-    frams_lib = FramsticksLib(parsed_args.path, parsed_args.lib,
-                              parsed_args.sim)
+    frams_lib = FramsticksLib(parsed_args.path, parsed_args.lib, parsed_args.sim)
 
     opt_dissim = []
@@ -303 +302 @@
     print("Running experiment with", sys.argv)
     parsed_args = parseArguments()
+    
+    if parsed_args.popsize % 4 != 0:
+        raise ValueError("popsize must be a multiple of 4 (for example %d)." % (parsed_args.popsize//4*4)) # required by deap.tools.selTournamentDCD() 
+        
     if parsed_args.debug:
         logging.basicConfig(level=logging.DEBUG)
@@ -308 +311 @@
     if parsed_args.checkpoint_path is not None and os.path.exists(parsed_args.checkpoint_path):
         experiment = load_experiment(parsed_args.checkpoint_path)
-        FramsticksLib(parsed_args.path, parsed_args.lib,
-                      parsed_args.sim)
+        FramsticksLib(parsed_args.path, parsed_args.lib, parsed_args.sim)
     else:
         experiment = create_experiment()

framspy/evolalg/selection/nsga2.py

@@ -11 +11 @@
         super(NSGA2Selection, self).call(population)
         pop = tools.selNSGA2(population, len(population))
-        # TODO make count divisible by 4, is this the best way to do it/is it absolutely required? if this is applied, why popsize must still be a multiple of 4?
+        # make count divisible by 4, required by deap.tools.selTournamentDCD()
         remainder=count % 4
         if remainder > 0: